Skip to content

Revert "[worker] feat: add support for colocate replicas"#4259

Merged
vermouth1992 merged 1 commit intomainfrom
revert-4233-colocate_use_rp
Nov 24, 2025
Merged

Revert "[worker] feat: add support for colocate replicas"#4259
vermouth1992 merged 1 commit intomainfrom
revert-4233-colocate_use_rp

Conversation

@wuxibin89
Copy link
Collaborator

Reverts #4233

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request reverts the feature 'add support for colocate replicas'. The changes correctly remove the logic related to collocated worker groups for reward models and restore the previous implementation that uses resource pools. The associated test file for the feature has also been removed. The revert appears to be complete and correct based on the provided changes. I have not identified any issues with this revert.

@vermouth1992 vermouth1992 merged commit 6a31dee into main Nov 24, 2025
60 of 65 checks passed
@vermouth1992 vermouth1992 deleted the revert-4233-colocate_use_rp branch November 24, 2025 07:17
TimurTaepov pushed a commit to giorgossideris/verl that referenced this pull request Dec 20, 2025
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