Skip to content

Conversation

pzhan9
Copy link
Contributor

@pzhan9 pzhan9 commented Oct 7, 2025

Differential Revision: D83838109

pzhan9 added 7 commits October 7, 2025 10:08
Differential Revision: D83931993
Differential Revision: D83934172
Differential Revision: D83530618
Differential Revision: D83533433
Summary: Pull Request resolved: meta-pytorch#1350

Differential Revision: D83385432
Differential Revision: D83591305
Summary:
Pull Request resolved: meta-pytorch#1304

This diff adds native v1 casting implementation, instead of piggybacking v0's implementation. Specifically, this diff:

1. Add V1 handlers in comm actors, i.e. `CastMessageV1` and `ForwardMessageV1`.
2. On the v1 ActorMesh side, use `Sequencers` to generate seq numbers, and send to comm actors through `CastMessageV1`.
3. Update several direct send call sites to use `ActorRef::seq_send`, so seq numbers are always assigned.

Note that we still need to wait for all `&impl cap::CanSend` callsites to be updated with `&impl context::Actor` before we can turn on the native v1 casting in production, since we need to ensure all `direct send`s are also getting the sequence number assigned.

Differential Revision: D82537988
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 7, 2025
Copy link

meta-codesync bot commented Oct 7, 2025

@pzhan9 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D83838109.

Differential Revision: D83839619
pzhan9 added a commit to pzhan9/monarch-1 that referenced this pull request Oct 8, 2025
Summary:
Pull Request resolved: meta-pytorch#1460

This diff enables the native v1 multicasting added in this stack for the python tests in `test_python_actors`.

Differential Revision: D83838109
Summary:
Pull Request resolved: meta-pytorch#1460

This diff enables the native v1 multicasting added in this stack for the python tests in `test_python_actors`.

Differential Revision: D83838109
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. fb-exported meta-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant