Skip to content

Conversation

@laithsakka
Copy link
Contributor

@laithsakka laithsakka commented Oct 11, 2024

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 11, 2024

🔗 Helpful Links

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

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

✅ No Failures

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

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

laithsakka added a commit that referenced this pull request Oct 11, 2024
ghstack-source-id: 839ad68
Pull Request resolved: #137760
@laithsakka laithsakka changed the title do not compute base in export mode Do not compute base in export mode Oct 11, 2024
@laithsakka laithsakka requested a review from zou3519 October 11, 2024 16:23
@laithsakka laithsakka added the topic: not user facing topic category label Oct 11, 2024
@laithsakka laithsakka requested a review from bdhirsh October 14, 2024 23:20
laithsakka added a commit that referenced this pull request Oct 14, 2024
ghstack-source-id: 42a24b3
Pull Request resolved: #137760

if (
torch.is_inference_mode_enabled()
not mode.export
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe worth a quick commenting mentioning it's because of a bug, not fundamental?

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.

5 participants