8000 [MRS-Inf] Deepcopy to avoid errors from inplace operators (#1245) by jiecaoyu · Pull Request #84020 · pytorch/pytorch · GitHub

[MRS-Inf] Deepcopy to avoid errors from inplace operators (#1245) #84020

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.

Sign up for GitHub

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

Jump to bottom

Closed

jiecaoyu wants to merge 1 commit into pytorch:master from jiecaoyu:export-D38716708

jiecaoyu commented

•

Summary: Pull Request resolved: pytorch/FBGEMM#1245

Reviewed By: houseroad

Differential Revision: D38716708

cc @ezyang @SherlockNoMad @soumith @EikanWang @jgong5 @wenzhe-nrv

facebook-github-bot added the cla signed label

Contributor

facebook-github-bot commented

•

🔗 Helpful links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/84020
✖️ Python docs build was skipped
✖️ C++ docs build was skipped
❓Need help or want to give feedback on the CI? Visit our office hours

✅ No Failures (1 Pending)

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

Expand to see more

💚 💚 Looks good so far! There are no failures yet. 💚 💚

This comment was automatically generated by Dr. CI (expand for details).

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

facebook-github-bot added fx fb-exported labels

Contributor

facebook-github-bot commented

This pull request was exported from Phabricator. Differential Revision: D38716708

1 similar comment

Contributor

facebook-github-bot commented

This pull request was exported from Phabricator. Differential Revision: D38716708

jiecaoyu force-pushed the export-D38716708 branch from 0d78bc0 to fdf6092 Compare

August 24, 2022 21:57

jiecaoyu added a commit to jiecaoyu/FBGEMM that referenced this pull request


          Deepcopy to avoid errors from inplace operators (#84020)

ee55938

Summary:
X-link: pytorch/pytorch#84020

Pull Request resolved: pytorch#1245

Reviewed By: houseroad

Differential Revision: D38716708

fbshipit-source-id: 19fa4903c5fd5deb937f9b759aafd9e009879397

jiecaoyu force-pushed the export-D38716708 branch from fdf6092 to c3dc407 Compare

August 25, 2022 07:44

Contributor

facebook-github-bot commented

This pull request was exported from Phabricator. Differential Revision: D38716708

1 similar comment

Contributor

facebook-github-bot commented

This pull request was exported from Phabricator. Differential Revision: D38716708

jiecaoyu force-pushed the export-D38716708 branch from c3dc407 to beb6b88 Compare

August 25, 2022 23:18

Contributor

facebook-github-bot commented

This pull request was exported from Phabricator. Differential Revision: D38716708

jiecaoyu force-pushed the export-D38716708 branch from beb6b88 to 3cc3b8d Compare

August 25, 2022 23:35

jiecaoyu force-pushed the export-D38716708 branch from 3cc3b8d to 048eff8 Compare

August 25, 2022 23:42

Contributor

facebook-github-bot commented

This pull request was exported from Phabricator. Differential Revision: D38716708


          [MRS-Inf] Deepcopy to avoid errors from inplace operators (pytorch#84020

2ceeb75

)

Summary:
Pull Request resolved: pytorch#84020

X-link: pytorch/FBGEMM#1245

Reviewed By: houseroad

Differential Revision: D38716708

fbshipit-source-id: 9ac2fae1e86687ae893054c039537846a3e2a2a9

jiecaoyu force-pushed the export-D38716708 branch from 048eff8 to 2ceeb75 Compare

August 25, 2022 23:49

Contributor

facebook-github-bot commented

This pull request was exported from Phabricator. Differential Revision: D38716708

Contributor

facebook-github-bot commented

/easycla

As part of the transition to the PyTorch Foundation, this project now requires contributions be covered under the new CLA. See #85559 for additional details.

This comment will trigger a new check of this PR. If you are already covered, you will simply see a new "EasyCLA" check that passes. If you are not covered, a bot will leave a new comment with a link to sign.

linux-foundation-easycla bot commented

❌ - login: @jiecaoyu / name: Jiecao Yu . The commit (2ceeb75) is not authorized under a signed CLA. Please click here to be authorized. For further assistance with EasyCLA, please submit a support request ticket.

Contributor

github-actions bot commented

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

github-actions bot added the Stale label

pytorch-bot bot added the release notes: fx label

github-actions bot closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed fb-exported fx release notes: fx Stale

0