Skip to content
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

Drop devmode, prepare RL.jl v0.11 for release #1035

Merged
merged 3 commits into from
Mar 11, 2024
Merged

Conversation

jeremiahpslewis
Copy link
Member

@jeremiahpslewis jeremiahpslewis commented Mar 8, 2024

To Do

  • Update docs and readme to reflect this change (and use the new RLCore api)
  • Drop RLExperiments / RLZoo from main docs

Description

  • tags a new RL.jl release
  • Drops devmode
  • Drops RLZoo from RL.jl dependencies
  • Adds RLFarm as new algorithm depot, empty for now
  • Going forward, RL.jl = Base + Env + Core, everything else is separate, should make breaking changes, etc. more manageable

@jeremiahpslewis jeremiahpslewis requested a review from HenriDeh March 8, 2024 16:02
Copy link

codecov bot commented Mar 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 24.05%. Comparing base (a6623a8) to head (9873141).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #1035       +/-   ##
===========================================
+ Coverage   12.74%   24.05%   +11.31%     
===========================================
  Files         229      230        +1     
  Lines        7903     7970       +67     
===========================================
+ Hits         1007     1917      +910     
+ Misses       6896     6053      -843     
Files Coverage Δ
...rcementLearningZoo/src/ReinforcementLearningZoo.jl 100.00% <ø> (ø)

... and 27 files with indirect coverage changes

@jeremiahpslewis
Copy link
Member Author

@HenriDeh would love your thoughts

HenriDeh
HenriDeh previously approved these changes Mar 11, 2024

This project contains updated, tested algorithms compatible with ReinforcementLearning.jl v0.11+.

Unlike ReinforcementLearningZoo, the algorithms here have been domesticated.
Copy link
Member

Choose a reason for hiding this comment

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

I love it

Comment on lines +9 to +12
else
using CUDA, cuDNN
CUDA.allowscalar(false)
end
Copy link
Member

Choose a reason for hiding this comment

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

Not a priority but there's also an AMD gpu support package now. I don't know if it is mature enough.

@jeremiahpslewis jeremiahpslewis merged commit 4f96c51 into main Mar 11, 2024
12 of 13 checks passed
@jeremiahpslewis jeremiahpslewis deleted the jpsl/rl-11 branch March 11, 2024 14:46
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