Skip to content

[jit] fix dictConstruct order issue#40424

Closed
wanchaol wants to merge 1 commit intogh/wanchaol/116/basefrom
gh/wanchaol/116/head
Closed

[jit] fix dictConstruct order issue#40424
wanchaol wants to merge 1 commit intogh/wanchaol/116/basefrom
gh/wanchaol/116/head

Conversation

@wanchaol
Copy link
Collaborator

@wanchaol wanchaol commented Jun 23, 2020

Stack from ghstack:

dictConstruct should preserve the inputs order

Differential Revision: D22202690

dictConstruct should preserve the inputs order

[ghstack-poisoned]
@wanchaol wanchaol requested a review from apaszke as a code owner June 23, 2020 07:45
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Jun 23, 2020
@dr-ci
Copy link

dr-ci bot commented Jun 23, 2020

💊 CI failures summary and remediations

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


  • 1/1 failures introduced in this PR

XLA failure

Job pytorch_xla_linux_bionic_py3_6_clang9_build 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 or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 1 time.

@wanchaol wanchaol requested review from eellison and jamesr66a June 23, 2020 17:04
@facebook-github-bot
Copy link
Contributor

@wanchaol merged this pull request in 787e1c4.

@facebook-github-bot facebook-github-bot deleted the gh/wanchaol/116/head branch June 28, 2020 14:17
wanchaol added a commit to wanchaol/pytorch that referenced this pull request Jun 30, 2020
A combination of pytorch#39601 and
pytorch#40424 both are approved and
merged in master
malfet pushed a commit that referenced this pull request Jul 1, 2020
A combination of #39601 and
#40424 both are approved and
merged in master
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