Skip to content

Conversation

@fegin
Copy link
Contributor

@fegin fegin commented Oct 29, 2025

Stack from ghstack (oldest at bottom):

pytorch/pytorch#166130 changes the configs and this PR adopts the new configs

Squash and Merge button won't work for this PR. I'll merge by myself.

[ghstack-poisoned]
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 29, 2025
Copy link
Member

@ruisizhang123 ruisizhang123 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you for the fix

@ruisizhang123 ruisizhang123 merged commit e295d04 into gh/fegin/21/base Oct 29, 2025
5 checks passed
fegin added a commit that referenced this pull request Oct 29, 2025
Stack from [ghstack](https://github.com/ezyang/ghstack/tree/0.12.0)
(oldest at bottom):
* #1960
* #1959
* __->__ #1965

**Squash and Merge button won't work for this PR. I'll merge by
myself.**

#1963 was accdientally merge
with Squash and Merge button. This is a reland PR.
jquesnelle pushed a commit to NousResearch/torchtitan that referenced this pull request Nov 10, 2025
Stack from [ghstack](https://github.com/ezyang/ghstack/tree/0.12.0)
(oldest at bottom):
* pytorch#1960
* pytorch#1959
* __->__ pytorch#1965

**Squash and Merge button won't work for this PR. I'll merge by
myself.**

pytorch#1963 was accdientally merge
with Squash and Merge button. This is a reland PR.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants