Skip to content

Fix the order of hosts in the ring#2718

Merged
angeloskath merged 1 commit intomainfrom
fix-dist-config-order
Oct 30, 2025
Merged

Fix the order of hosts in the ring#2718
angeloskath merged 1 commit intomainfrom
fix-dist-config-order

Conversation

@angeloskath
Copy link
Copy Markdown
Member

@angeloskath angeloskath commented Oct 30, 2025

mlx.distributed_config would always return the ring order passed in from the hosts argument instead of the ring order found from the TB connectivity.

@thechriswebb that should also remove the need of passing the hosts in the right order.

Copy link
Copy Markdown
Member

@awni awni left a comment

Choose a reason for hiding this comment

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

Thanks!

@angeloskath angeloskath merged commit b901a9f into main Oct 30, 2025
5 checks passed
@angeloskath angeloskath deleted the fix-dist-config-order branch October 30, 2025 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants