Skip to content

Conversation

@shayne-fletcher
Copy link
Contributor

Differential Revision: D88401829

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Dec 4, 2025
@meta-codesync
Copy link

meta-codesync bot commented Dec 4, 2025

@shayne-fletcher has exported this pull request. If you are a Meta employee, you can view the originating Diff in D88401829.

shayne-fletcher added a commit to shayne-fletcher/monarch-1 that referenced this pull request Dec 4, 2025
Summary: Pull Request resolved: meta-pytorch#2048

Differential Revision: D88401829
Summary:

this change removes the dependency on hyperactor_multiprocess from the monarch_hyperactor crate with the smallest possible surface-area change.

- remove hyperactor_multiprocess from BUCK and Cargo.toml so the python bindings no longer depend on the multiprocess/system-actor stack.
- delete python-facing helpers that were thin wrappers over the system actor and proc supervision path:
  - `PyActor.world_status`
  - `PyProc.world_status`
  - `check_actor_supervision_state` and the periodic controller health check
- replace the multiprocess-based proc bootstrap in `init_proc` with `unimplemented!()`, so the symbol still exists but no longer wires a `Proc` into the old system-actor supervision model.

the intent here is just to decouple monarch_hyperactor from hyperactor_multiprocess; no new bootstrap or supervision path is introduced. any remaining callers that depend on `init_proc` or `world_status` will now hit `unimplemented!()` at runtime and should be migrated to the newer mesh/host/proc bootstrap APIs in subsequent stacks.

Differential Revision: D88401829
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