Skip to content

[jit] Add dictionary as output of tracer#36696

Closed
wanchaol wants to merge 3 commits intogh/wanchaol/99/basefrom
gh/wanchaol/99/head
Closed

[jit] Add dictionary as output of tracer#36696
wanchaol wants to merge 3 commits intogh/wanchaol/99/basefrom
gh/wanchaol/99/head

Conversation

@wanchaol
Copy link
Copy Markdown
Collaborator

@wanchaol wanchaol commented Apr 16, 2020

Stack from ghstack:

This PR add dictionary as a supported output of tracer under the strict
flag.

Differential Revision: D21056962

This PR add dictionary as a supported output of tracer under the strict
flag.

[ghstack-poisoned]
@wanchaol wanchaol requested a review from apaszke as a code owner April 16, 2020 00:56
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Apr 16, 2020
@dr-ci
Copy link
Copy Markdown

dr-ci Bot commented Apr 16, 2020

💊 Build failures summary and remediations

As of commit 2aaf04e (more details on the Dr. CI page):


  • 1/1 failures introduced in this PR

XLA failure

Job pytorch_xla_linux_bionic_py3_6_clang9_test is failing. Please create an issue with title prefixed by [PT_BREAK] in pytorch/xla and link to to this PR. If you have questions, please reach out to @ailzhang / @dlibenzi / @JackCaoG.


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.

See how this bot performed.

This comment has been revised 9 times.

Copy link
Copy Markdown
Collaborator

@jamesr66a jamesr66a left a comment

Choose a reason for hiding this comment

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

Great, thanks for hanging in there as we resolved the ordering issues. I'm glad we were able to solve a hidden bug in TorchScript as a side effect of this

This PR add dictionary as a supported output of tracer under the strict
flag.

Differential Revision: [D21056962](https://our.internmc.facebook.com/intern/diff/D21056962)

[ghstack-poisoned]
wanchaol added a commit that referenced this pull request Apr 16, 2020
This PR add dictionary as a supported output of tracer under the strict
flag.

ghstack-source-id: 5c68822
Pull Request resolved: #36696
This PR add dictionary as a supported output of tracer under the strict
flag.

Differential Revision: [D21056962](https://our.internmc.facebook.com/intern/diff/D21056962)

[ghstack-poisoned]
wanchaol added a commit that referenced this pull request Apr 16, 2020
This PR add dictionary as a supported output of tracer under the strict
flag.

ghstack-source-id: ca2405f
Pull Request resolved: #36696
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@wanchaol merged this pull request in 24aac32.

@facebook-github-bot facebook-github-bot deleted the gh/wanchaol/99/head branch April 20, 2020 14:17
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#36696

This PR add dictionary as a supported output of tracer under the strict
flag.

Test Plan: Imported from OSS

Reviewed By: houseroad

Differential Revision: D21056962

Pulled By: wanchaol

fbshipit-source-id: ace498182d636de853cf8a1efb3dc77f5d53db29
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