Skip to content

Conversation

gmagogsfm
Copy link
Contributor

@gmagogsfm gmagogsfm commented Aug 11, 2020

Stack from ghstack:

Differential Revision: D23222894

@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Aug 11, 2020
gmagogsfm added a commit that referenced this pull request Aug 11, 2020
@dr-ci
Copy link

dr-ci bot commented Aug 11, 2020

💊 CI failures summary and remediations

As of commit 555e352 (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 8 times.

Copy link

@SplitInfinity SplitInfinity left a comment

Choose a reason for hiding this comment

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

What is the motivation for this PR?

I worry that the IR representation of Enums is an internal implementation detail that shouldn't matter much as long as enums behave as they should (which we already have tests for).

@gmagogsfm
Copy link
Contributor Author

What is the motivation for this PR?

I worry that the IR representation of Enums is an internal implementation detail that shouldn't matter much as long as enums behave as they should (which we already have tests for).

It is technically internal implementation detail from end-user's perspective. But I think it is still worth testing that key parts of IR is generated correctly. These filechecks are cheap and key parts (type annotation, how constants are printed etc) shouldn't change that much.

@facebook-github-bot
Copy link
Contributor

@gmagogsfm merged this pull request in 8eb3de7.

@facebook-github-bot facebook-github-bot deleted the gh/gmagogsfm/3/head branch August 25, 2020 14:14
facebook-github-bot pushed a commit that referenced this pull request Sep 3, 2020
Summary:
Stack from [ghstack](https://github.com/ezyang/ghstack):
* **#43188 Enable Enum pickling/unpickling.**
* #42963 Add Enum TorchScript serialization and deserialization support
* #42874 Fix enum constant printing and add FileCheck to all Enum tests
* #43121 Add Enum convert back to Python object support

Pull Request resolved: #43188

Reviewed By: zdevito

Differential Revision: D23365141

Pulled By: gmagogsfm

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

Labels

Merged oncall: jit Add this issue/PR to JIT oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants