KEMBAR78
[dtensor] enable adagrad foreach support by wanchaol · Pull Request #114151 · pytorch/pytorch · GitHub
Skip to content

Conversation

@wanchaol
Copy link
Collaborator

@wanchaol wanchaol commented Nov 20, 2023

This PR enables the adagrad foreach mode support

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Nov 20, 2023

🔗 Helpful Links

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

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

✅ No Failures

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

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

Copy link
Contributor

@XilunWu XilunWu left a comment

Choose a reason for hiding this comment

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

LGTM! 2 suggestions

This PR enables the adagrad foreach mode support

cc H-Huang awgu kwen2501 fegin fduwjj wz337 wconstab mrshenli zhaojuanmao rohan-varma kiukchung d4l3k lucasllc XilunWu

[ghstack-poisoned]
@XilunWu
Copy link
Contributor

XilunWu commented Nov 20, 2023

Thanks for addressing the comments. I think we're good to go!

pytorchmergebot pushed a commit that referenced this pull request Nov 21, 2023
@facebook-github-bot facebook-github-bot deleted the gh/wanchaol/396/head branch November 24, 2023 15:28
@albanD albanD added oncall: distributed Add this issue/PR to distributed oncall triage queue and removed module: distributed labels Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged oncall: distributed Add this issue/PR to distributed oncall triage queue release notes: distributed (dtensor) release notes category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants