Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ONNX Export IsNan op#17698

Closed
lara-hdr wants to merge 12 commits into
pytorch:masterfrom
lara-hdr:lahaidar/onnx_export_isnan
Closed

ONNX Export IsNan op#17698
lara-hdr wants to merge 12 commits into
pytorch:masterfrom
lara-hdr:lahaidar/onnx_export_isnan

Conversation

@lara-hdr
Copy link
Copy Markdown
Contributor

@lara-hdr lara-hdr commented Mar 5, 2019

No description provided.

@ezyang ezyang requested a review from houseroad March 6, 2019 18:15
Comment thread test/onnx/test_pytorch_onnx_caffe2.py Outdated
Comment thread caffe2/operators/utility_ops.cc Outdated
Comment thread caffe2/operators/utility_ops.h Outdated
@houseroad
Copy link
Copy Markdown
Member

@lara-hdr to pass the CI, you need to rebase to a newer commit on master

Copy link
Copy Markdown
Member

@houseroad houseroad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but please rebase the PR, and trigger the CI again.

@ezyang
Copy link
Copy Markdown
Contributor

ezyang commented Mar 11, 2019

@pytorchbot rebase this please

@houseroad
Copy link
Copy Markdown
Member

Oh, this is a nice feature !

@lara-hdr
Copy link
Copy Markdown
Contributor Author

@houseroad all tests are passing with the rebase

@lara-hdr
Copy link
Copy Markdown
Contributor Author

@houseroad , can we merge this? :)

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@houseroad has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Copy Markdown
Member

@houseroad houseroad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks!

@zrphercule zrphercule self-requested a review March 15, 2019 18:18
Copy link
Copy Markdown
Contributor

@zrphercule zrphercule left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, great job!

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary: Pull Request resolved: pytorch#17698

Reviewed By: zrphercule

Differential Revision: D14470646

Pulled By: houseroad

fbshipit-source-id: d3e6adc83c4f9fa288c5fe0ae4c6af71fdd47905
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants