KEMBAR78
[Workspace Inductor] Fix dynamic shapes by drisspg · Pull Request #139777 · pytorch/pytorch · GitHub
Skip to content

Conversation

@drisspg
Copy link
Contributor

@drisspg drisspg commented Nov 5, 2024

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Nov 5, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 7d5df77 with merge base 8881108 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@drisspg drisspg requested a review from eellison November 5, 2024 18:41
@drisspg drisspg added the topic: not user facing topic category label Nov 5, 2024
@drisspg drisspg requested review from Chillee and ezyang November 5, 2024 18:42
@drisspg drisspg changed the title [Workspace Inductor] Fix dyanmic shapes [Workspace Inductor] Fix dynamic shapes Nov 5, 2024
@drisspg
Copy link
Contributor Author

drisspg commented Nov 5, 2024

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 5, 2024
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

pobin6 pushed a commit to pobin6/pytorch that referenced this pull request Dec 5, 2024
# Summary
Arg ordering was wrong for when dynamic shapes is enabled and we pass in the additional size args

Pull Request resolved: pytorch#139777
Approved by: https://github.com/eellison
ghstack dependencies: pytorch#139157
@github-actions github-actions bot deleted the gh/drisspg/80/head branch December 6, 2024 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants