Skip to content

Conversation

@eellison
Copy link
Contributor

@eellison eellison commented Oct 9, 2020

Fixes for #45627
Merged Prs: #45857. #45929

Fixes silent correctness issue - bug with dict constant hashing, and bug with not over writing existing key with dict.update.

@eellison eellison requested a review from apaszke as a code owner October 9, 2020 19:13
@eellison eellison changed the title Dict fixes2 [JIT] Dict Bug Fixes Oct 9, 2020
@facebook-github-bot facebook-github-bot added the oncall: jit Add this issue/PR to JIT oncall triage queue label Oct 9, 2020
@gchanan gchanan added this to the 1.7.0 milestone Oct 9, 2020
@gchanan gchanan changed the title [JIT] Dict Bug Fixes [v1.7] [JIT] Dict Bug Fixes Oct 9, 2020
@gchanan gchanan changed the title [v1.7] [JIT] Dict Bug Fixes [v1.7 cherry-pick] [JIT] Dict Bug Fixes Oct 9, 2020
@dr-ci
Copy link

dr-ci bot commented Oct 9, 2020

💊 CI failures summary and remediations

As of commit 2520016 (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 1 time.

@codecov
Copy link

codecov bot commented Oct 9, 2020

Codecov Report

Merging #46105 into release/1.7 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/1.7   #46105   +/-   ##
============================================
  Coverage        68.46%   68.46%           
============================================
  Files              406      406           
  Lines            52267    52267           
============================================
  Hits             35785    35785           
  Misses           16482    16482           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e93ae1e...2520016. Read the comment docs.

@malfet malfet merged commit 36fe788 into release/1.7 Oct 12, 2020
@facebook-github-bot facebook-github-bot deleted the dict_fixes2 branch January 27, 2021 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants