Skip to content

[canary] force_nn_module_property_static_shapes=False #148139

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 2 commits into from

Conversation

bobrenjc93
Copy link
Contributor

@bobrenjc93 bobrenjc93 commented Feb 28, 2025

Stack from ghstack (oldest at bottom):

As part of the dynamic shapes roadmap this half, we want to reduce the number of unrolled out flags. This is one that limits dynamism and doesn't seem to affect compile time or correctness. Let's flip it to False by default.

Copy link

pytorch-bot bot commented Feb 28, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit bfae592 with merge base 5220d40 (image):
💚 Looks good so far! There are no failures yet. 💚

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

bobrenjc93 added a commit that referenced this pull request Feb 28, 2025
ghstack-source-id: 749e913
Pull Request resolved: #148139
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]
bobrenjc93 added a commit that referenced this pull request Feb 28, 2025
ghstack-source-id: 3d9a610
Pull Request resolved: #148139
@bobrenjc93 bobrenjc93 added the topic: not user facing topic category label Feb 28, 2025
@bobrenjc93
Copy link
Contributor Author

@bobrenjc93
Copy link
Contributor Author

Screenshot 2025-03-01 at 1 24 25 PM

Surprisingly lots of improvement in compile times

@bobrenjc93 bobrenjc93 requested a review from laithsakka March 1, 2025 21:25
@bobrenjc93 bobrenjc93 marked this pull request as ready for review March 1, 2025 21:26
@bobrenjc93 bobrenjc93 closed this Mar 4, 2025
@github-actions github-actions bot deleted the gh/bobrenjc93/292/head branch April 4, 2025 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant