-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[RLlib] Learner API: Policies using RLModules (for sampler only) do not need loss/stats/mixins. #34445
Merged
sven1977
merged 41 commits into
ray-project:master
from
sven1977:learner_rlm_policies_simplifications
May 8, 2023
Merged
[RLlib] Learner API: Policies using RLModules (for sampler only) do not need loss/stats/mixins. #34445
Changes from 1 commit
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
47c003c
wip
sven1977 cc25d0c
wip
sven1977 e6f2d3b
merge
sven1977 0248ce8
wip
sven1977 04595b3
wip
sven1977 bf78988
wip
sven1977 ceb8caa
wip
sven1977 cc632cb
Merge branch 'master' of https://github.com/ray-project/ray into lear…
sven1977 904bfda
merge
sven1977 b36304d
wip
sven1977 a112d5b
wip
sven1977 4b600ef
wip
sven1977 6dba48b
wip
sven1977 95fd464
wip
sven1977 6e8ab3f
wip
sven1977 b224d9f
wip
sven1977 26bdf8d
fix
sven1977 07409be
fix
sven1977 74f159f
wip
sven1977 70e6127
fix
sven1977 e4a58ee
fix
sven1977 c40cb8d
fix
sven1977 a2bc97a
fix
sven1977 81bcbd5
fix
sven1977 499d818
wip
sven1977 fd61ef6
LINT
sven1977 a53b043
fix
sven1977 f175a39
LINT
sven1977 3e95159
fix
sven1977 0ad186f
fix
sven1977 3a85eae
wip
sven1977 1380cb6
Merge remote-tracking branch 'origin/learner_rlm_policies_simplificat…
sven1977 e9ad050
merge
sven1977 05103ba
fix
sven1977 f0c1145
LINT
sven1977 90ee0ab
Add new Scheduler API.
sven1977 a628043
Merge branch 'master' of https://github.com/ray-project/ray into lear…
sven1977 6e9b0cd
wip
sven1977 7f85d0a
LINT
sven1977 1bb3ec2
fix
sven1977 21486d7
Merge branch 'master' of https://github.com/ray-project/ray into lear…
sven1977 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
fix
Signed-off-by: sven1977 <[email protected]>
- Loading branch information
commit 81bcbd5f420b0cc629489f93e0ceb27cfb076888
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had to fix a bunch of test cases that were still using learner=False, but RLModule=True. This is not possible anymore.