Skip to content

Conversation

pianpwk
Copy link
Contributor

@pianpwk pianpwk commented Oct 2, 2025

Copy link

pytorch-bot bot commented Oct 2, 2025

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit a3ed4e4 with merge base 6a31f42 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@pianpwk pianpwk changed the title [WIP] fix multikernel shape keys [multi-kernel] base tensor sizes for shape cache key Oct 2, 2025
@pianpwk pianpwk marked this pull request as ready for review October 2, 2025 20:43
@ColinPeppler
Copy link
Contributor

I was thinking what happens if we have a base tensor that is dynamic, but we take a view with static shapes, will this cause _kernel_has_dynamic_shapes to return True when it shouldn't have?

@pianpwk
Copy link
Contributor Author

pianpwk commented Oct 7, 2025

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Oct 7, 2025
@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

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