Skip to content

[map] always turn on dynamo for map #150962

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.

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

Closed
wants to merge 8 commits into from
Closed

Conversation

@ydwu4 ydwu4 requested a review from zou3519 as a code owner April 9, 2025 21:42
Copy link

pytorch-bot bot commented Apr 9, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/150962

Note: Links to docs will display an error until the docs builds have been completed.

✅ You can merge normally! (2 Unrelated Failures)

As of commit a378223 with merge base 1250106 (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

ydwu4 added a commit that referenced this pull request Apr 9, 2025
ghstack-source-id: 4fcbbc6
Pull Request resolved: #150962
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
ydwu4 added a commit that referenced this pull request Apr 9, 2025
ghstack-source-id: 0576853
Pull Request resolved: #150962
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
@ydwu4 ydwu4 added the keep-going Don't stop on first failure, keep running tests until the end label Apr 10, 2025
ydwu4 added 2 commits April 10, 2025 13:47
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
@ydwu4 ydwu4 added the topic: not user facing topic category label Apr 10, 2025
ydwu4 added 3 commits April 11, 2025 09:35
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
@ydwu4 ydwu4 requested review from ezyang and Chillee as code owners April 11, 2025 18:17
@pytorchmergebot
Copy link
Collaborator

Starting merge as part of PR stack under #151034

1 similar comment
@pytorchmergebot
Copy link
Collaborator

Starting merge as part of PR stack under #151034

pytorchmergebot pushed a commit that referenced this pull request Apr 11, 2025
@Camyll
Copy link
Contributor

Camyll commented Apr 14, 2025

@pytorchbot revert

Copy link

pytorch-bot bot commented Apr 14, 2025

❌ 🤖 pytorchbot command failed:

@pytorchbot revert: error: the following arguments are required: -m/--message, -c/--classification

usage: @pytorchbot revert -m MESSAGE -c
                          {nosignal,ignoredsignal,landrace,weird,ghfirst}

Try @pytorchbot --help for more info.

@Camyll
Copy link
Contributor

Camyll commented Apr 14, 2025

@pytorchbot revert -m "breaking internal builds by failing 4 tests" -c ghfirst

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

pytorchmergebot added a commit that referenced this pull request Apr 14, 2025
This reverts commit ca2e8cd.

Reverted #151034 on behalf of https://github.com/Camyll due to breaking internal builds {SHORT_REASON} ([comment](#150962 (comment)))
pytorchmergebot added a commit that referenced this pull request Apr 14, 2025
This reverts commit a72d56c.

Reverted #150962 on behalf of https://github.com/Camyll due to breaking internal builds {SHORT_REASON} ([comment](#150962 (comment)))
@pytorchmergebot
Copy link
Collaborator

@ydwu4 your PR has been successfully reverted.

@pytorchmergebot pytorchmergebot added Reverted ci-no-td Do not run TD on this PR labels Apr 14, 2025
timocafe pushed a commit to timocafe/pytorch that referenced this pull request Apr 16, 2025
timocafe pushed a commit to timocafe/pytorch that referenced this pull request Apr 16, 2025
timocafe pushed a commit to timocafe/pytorch that referenced this pull request Apr 16, 2025
This reverts commit ca2e8cd.

Reverted pytorch#151034 on behalf of https://github.com/Camyll due to breaking internal builds {SHORT_REASON} ([comment](pytorch#150962 (comment)))
timocafe pushed a commit to timocafe/pytorch that referenced this pull request Apr 16, 2025
This reverts commit a72d56c.

Reverted pytorch#150962 on behalf of https://github.com/Camyll due to breaking internal builds {SHORT_REASON} ([comment](pytorch#150962 (comment)))
amathewc pushed a commit to amathewc/pytorch that referenced this pull request Apr 17, 2025
amathewc pushed a commit to amathewc/pytorch that referenced this pull request Apr 17, 2025
amathewc pushed a commit to amathewc/pytorch that referenced this pull request Apr 17, 2025
This reverts commit ca2e8cd.

Reverted pytorch#151034 on behalf of https://github.com/Camyll due to breaking internal builds {SHORT_REASON} ([comment](pytorch#150962 (comment)))
amathewc pushed a commit to amathewc/pytorch that referenced this pull request Apr 17, 2025
This reverts commit a72d56c.

Reverted pytorch#150962 on behalf of https://github.com/Camyll due to breaking internal builds {SHORT_REASON} ([comment](pytorch#150962 (comment)))
Divigroup-RAP pushed a commit to Divigroup-RAP/PYTORCH that referenced this pull request Apr 22, 2025
ghstack-source-id: e21aeb5
Pull Request resolved: pytorch/pytorch#150962
ydwu4 added a commit to ydwu4/pytorch that referenced this pull request Apr 23, 2025
Summary:
X-link: pytorch/executorch#10409

Reland D72896450

Make map consistent with other control flow ops. After the change, map is able to support accessing closures in the map fn.

Test Plan: See existing tests.

Reviewed By: zou3519

Differential Revision: D73138427
pytorch-bot bot pushed a commit that referenced this pull request May 2, 2025
Summary:

X-link: pytorch/executorch#10409

Reland D72896450

Make map consistent with other control flow ops. After the change, map is able to support accessing closures in the map fn.

Test Plan: See existing tests.

Reviewed By: zou3519, angelayi

Differential Revision: D73138427
ydwu4 added a commit to ydwu4/pytorch that referenced this pull request May 2, 2025
Summary:


Reland D72896450

Make map consistent with other control flow ops. After the change, map is able to support accessing closures in the map fn.

Test Plan: See existing tests.

Reviewed By: zou3519, angelayi

Differential Revision: D73138427
ydwu4 added a commit to ydwu4/pytorch that referenced this pull request May 2, 2025
Summary:


Reland D72896450

Make map consistent with other control flow ops. After the change, map is able to support accessing closures in the map fn.

Test Plan: See existing tests.

Reviewed By: zou3519, angelayi

Differential Revision: D73138427
pytorch-bot bot pushed a commit that referenced this pull request May 5, 2025
Summary:


Reland D72896450

Make map consistent with other control flow ops. After the change, map is able to support accessing closures in the map fn.

Test Plan: See existing tests.

Reviewed By: zou3519, angelayi

Differential Revision: D73138427
@ydwu4
Copy link
Contributor Author

ydwu4 commented May 5, 2025

Replaced by #152041

@ydwu4 ydwu4 closed this May 5, 2025
pytorch-bot bot pushed a commit that referenced this pull request May 7, 2025
Summary:


Reland D72896450

Make map consistent with other control flow ops. After the change, map is able to support accessing closures in the map fn.

Test Plan: See existing tests.

Reviewed By: zou3519, angelayi

Differential Revision: D73138427
@github-actions github-actions bot deleted the gh/ydwu4/229/head branch June 7, 2025 02:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-no-td Do not run TD on this PR ciflow/inductor keep-going Don't stop on first failure, keep running tests until the end Merged module: dynamo Reverted topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants