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

Remove stable baselines3 #18

Merged
merged 3 commits into from
Sep 13, 2022
Merged

Remove stable baselines3 #18

merged 3 commits into from
Sep 13, 2022

Conversation

verdimrc
Copy link
Contributor

Issue #, if available: close #16

Description of changes: Make stable_baselines3 part of requirements-docs.txt only.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@github-actions
Copy link

github-actions bot commented Sep 11, 2022

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/a2rl
   __init__.py20195%34
   simulator.py5775690%456–458, 471, 537, 543, 559–594, 600, 912, 1011, 1026, 1036, 1041, 1057, 1075, 1097, 1123, 1144, 1150, 1167, 1171, 1177, 1187, 1194, 1217, 1223, 1302, 1309, 1364, 1371, 1377, 1384, 1387, 1498, 1514–1516, 1525, 1549, 1588, 1591
   tokenizer.py116298%64–65
   utils.py1612286%51, 60–63, 74–76, 108, 147, 164, 179–181, 336–339, 475–484, 544, 558
src/a2rl/experimental/lightgpt
   lr_decay.py20195%30
   model.py116298%260, 263
   simulator.py35197%162
src/a2rl/mingpt
   model.py118497%57, 203, 208, 230
   trainer.py841483%50–51, 55–57, 108–111, 116, 124–126, 134, 140–141
TOTAL151010393% 

Tests Skipped Failures Errors Time
230 0 💤 0 ❌ 0 🔥 36.450s ⏱️

Copy link
Contributor

@yapweiyih yapweiyih left a comment

Choose a reason for hiding this comment

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

Approved

@verdimrc verdimrc self-assigned this Sep 13, 2022
@verdimrc verdimrc merged commit 4a3815b into main Sep 13, 2022
@verdimrc verdimrc deleted the remove-stable-baselines3 branch September 13, 2022 05:59
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.

Remove dependency to stable_baseline3
2 participants