Skip to content

(3/n) Specialize state actor bootstrap for each alloc #459

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

highker
Copy link

@highker highker commented Jul 8, 2025

Summary:
Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 8, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

@highker highker force-pushed the export-D77914042 branch from c010d74 to daaee6d Compare July 9, 2025 06:43
highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch 2 times, most recently from 78b0474 to bc43c95 Compare July 9, 2025 18:32
highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from bc43c95 to 744ba5d Compare July 9, 2025 18:37
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from 744ba5d to bb10c1b Compare July 9, 2025 18:46
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from bb10c1b to df14fc1 Compare July 10, 2025 18:44
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from df14fc1 to de13844 Compare July 10, 2025 18:45
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from de13844 to be7599e Compare July 10, 2025 18:49
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

@highker highker force-pushed the export-D77914042 branch from be7599e to 96b1487 Compare July 10, 2025 19:01
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants