Skip to content

Revert "Revert "[RLlib] AlphaStar: Parallelized, multi-agent/multi-GPU learni…"#22153

Merged
sven1977 merged 7 commits intomasterfrom
revert-22105-revert-rllib-failure
Feb 8, 2022
Merged

Revert "Revert "[RLlib] AlphaStar: Parallelized, multi-agent/multi-GPU learni…"#22153
sven1977 merged 7 commits intomasterfrom
revert-22105-revert-rllib-failure

Conversation

@sven1977
Copy link
Copy Markdown
Contributor

@sven1977 sven1977 commented Feb 6, 2022

Let's try this again, this time, watching serve and other libraries' tests more closely. RLlib tests seemed to be all ok.

Reverts #22105

@avnishn
Copy link
Copy Markdown
Member

avnishn commented Feb 6, 2022

Same error with rllib imports popping up :(

@sven1977 sven1977 added the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Feb 8, 2022
@sven1977
Copy link
Copy Markdown
Contributor Author

sven1977 commented Feb 8, 2022

@avnishn, @gjoliver , this is fixed now. Could you re-approve? Thanks!

@sven1977 sven1977 merged commit c17a44c into master Feb 8, 2022
@sven1977 sven1977 deleted the revert-22105-revert-rllib-failure branch February 8, 2022 15:43
simonsays1980 pushed a commit to simonsays1980/ray that referenced this pull request Feb 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests-ok The tagger certifies test failures are unrelated and assumes personal liability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants