Skip to content

Multinode fix, adds latency tracker #25

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 7 commits into from
Sep 18, 2025
Merged

Multinode fix, adds latency tracker #25

merged 7 commits into from
Sep 18, 2025

Conversation

@LucasLLC
Copy link
Contributor

@LucasLLC LucasLLC commented Sep 8, 2025 •
edited
Loading

  • Adds 'latency tracker' to better understand cost of put/get api's.
  • Latency tracker lead me to realize one small bug which was sending a tensor unnecessarily through monarch comms -- fixed
  • Adds support for passing a process mesh to torchstore.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Sep 8, 2025
@LucasLLC LucasLLC changed the title [WIP] Prevent entire tensor fetch on resharding logic Multinode support Sep 10, 2025
@LucasLLC LucasLLC changed the title Multinode support Multinode fix, adds latency tracker Sep 10, 2025
@LucasLLC LucasLLC marked this pull request as ready for review September 10, 2025 20:15
@LucasLLC LucasLLC merged commit 2e44a34 into main Sep 18, 2025
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pradeepfn pradeepfn pradeepfn approved these changes

@pbontrager pbontrager pbontrager approved these changes

@allenwang28 allenwang28 Awaiting requested review from allenwang28

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@LucasLLC @pradeepfn @pbontrager