Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ONNX Export Support for Celu #38243

Closed
wants to merge 10 commits into from
Closed

ONNX Export Support for Celu #38243

wants to merge 10 commits into from

Conversation

KsenijaS
Copy link
Contributor

Add ONNX export support for torch.nn.CELU

@dr-ci
Copy link

dr-ci bot commented May 11, 2020

💊 CI failures summary and remediations

As of commit 4ade58e (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)

ci.pytorch.org: 1 failed


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 23 times.

@ailzhang ailzhang added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label May 12, 2020
Copy link
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
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
Contributor

@neginraoof neginraoof left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@KsenijaS
Copy link
Contributor Author

@houseroad please review

@KsenijaS
Copy link
Contributor Author

@houseroad for review. Thanks!

@ezyang ezyang removed the request for review from houseroad May 26, 2020 20:04
@ezyang ezyang removed the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label May 26, 2020
Copy link
Contributor

@ezyang ezyang left a comment

Choose a reason for hiding this comment

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

@KsenijaS please go ahead and land this

@KsenijaS
Copy link
Contributor Author

@ezyang I don't think I have privileges to land a PR.

@KsenijaS
Copy link
Contributor Author

@ezyang can you please merge this PR. Thanks!

@ezyang
Copy link
Contributor

ezyang commented May 27, 2020

Oh, @houseroad imported it, I'll let him handle it

@KsenijaS
Copy link
Contributor Author

@houseroad can you please merge this PR. Thanks!

Copy link
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.

@houseroad
Copy link
Member

Sure, will land it soon.

@facebook-github-bot
Copy link
Contributor

@houseroad merged this pull request in 858ab75.

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.

None yet

9 participants