Skip to content

Conversation

amirafzali
Copy link
Member

@amirafzali amirafzali commented Oct 13, 2025

Description

This adds a tutorial demonstrating how to use Monarch with TorchTitan to spin up distributed jobs from a single controller.

We would like to link to this as part of the Monarch pytorch.org blog post.

Copy link

pytorch-bot bot commented Oct 13, 2025

🔗 Helpful Links

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

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

❗ 2 Active SEVs

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

✅ No Failures

As of commit 3a7f607 with merge base 3469d47 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@meta-cla meta-cla bot added the cla signed label Oct 13, 2025
@amirafzali amirafzali marked this pull request as ready for review October 13, 2025 14:37
Copy link
Member

@dcci dcci left a comment

Choose a reason for hiding this comment

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

Approved, but please wait for @svekars to take a look at the comment in case she has anything.

Copy link
Contributor

@svekars svekars left a comment

Choose a reason for hiding this comment

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

Added a couple of minor comments but LGTM overall.
One question - we don't want this to be executable, right? and have a notebook?

@amirafzali
Copy link
Member Author

Added a couple of minor comments but LGTM overall. One question - we don't want this to be executable, right? and have a notebook?

Thanks for the quick review! Addressed your feedback. There's a notebook linked at the bottom of the tutorial

@amirafzali amirafzali force-pushed the monarch-distributed-tutorial branch 2 times, most recently from 39c7dcb to 479d02d Compare October 13, 2025 22:36
@amirafzali amirafzali force-pushed the monarch-distributed-tutorial branch from 479d02d to 1a58734 Compare October 13, 2025 22:39
@amirafzali amirafzali merged commit 2360b06 into main Oct 14, 2025
21 checks passed
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.

4 participants