-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix tests for mps support #1
base: master
Are you sure you want to change the base?
Changes from all commits
ace0516
9ac6225
2dcbef9
b00ca7f
06a2124
6d868c0
8d79e96
3276cb0
f4f6073
64327c7
0344c3c
fa196ab
efd086e
7f11843
c60f681
92e8d11
b235c8e
d4d0536
0311b62
086f79a
fe606fc
34f4819
ef39571
d26324c
1e5dc90
40ed03c
e83924b
b707480
81e3c63
f0e54a7
d47c586
b85a2a5
1c25053
f822ef5
1ac4a60
9970f51
955382e
56c153f
3d59b5c
dd3d0ac
5e7372d
263e657
8f0b488
e4f4f12
7c71688
4c03a25
020ee42
daaebd0
9489b1a
0ec37d8
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,19 @@ | ||
name: root | ||
channels: | ||
- pytorch | ||
- defaults | ||
- conda-forge | ||
dependencies: | ||
- cpuonly=1.0=0 | ||
- pip=22.3.1 | ||
- python=3.8 | ||
- pytorch=1.13.0=py3.8_cpu_0 | ||
- pip=24.2 | ||
- python=3.11 | ||
- pytorch=2.5.0=py3.11_cpu_0 | ||
- pip: | ||
- gymnasium | ||
- gymnasium>=0.29.1,<1.1.0 | ||
- cloudpickle | ||
- opencv-python-headless | ||
- pandas | ||
- numpy | ||
- numpy>=1.20,<3.0 | ||
- matplotlib | ||
- sphinx>=5,<8 | ||
- sphinx>=5,<9 | ||
- sphinx_rtd_theme>=1.3.0 | ||
- sphinx_copybutton |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,6 +10,7 @@ Name ``Box`` ``Discrete`` ``MultiDiscrete`` ``MultiBinary`` | |
=================== =========== ============ ================= =============== ================ | ||
ARS [#f1]_ ✔️ ✔️ ❌ ❌ ✔️ | ||
A2C ✔️ ✔️ ✔️ ✔️ ✔️ | ||
CrossQ [#f1]_ ✔️ ❌ ❌ ❌ ✔️ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Inconsistency in CrossQ implementation location. There appears to be an inconsistency in the documentation. The table indicates CrossQ is implemented in SB3 Contrib (via footnote [#f1]), but the note at the bottom of the file mentions it's in the SBX repo. Please clarify which repository actually contains the CrossQ implementation. |
||
DDPG ✔️ ❌ ❌ ❌ ✔️ | ||
DQN ❌ ✔️ ❌ ❌ ✔️ | ||
HER ✔️ ✔️ ❌ ❌ ✔️ | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,7 +7,7 @@ Installation | |
Prerequisites | ||
------------- | ||
|
||
Stable-Baselines3 requires python 3.8+ and PyTorch >= 1.13 | ||
Stable-Baselines3 requires python 3.9+ and PyTorch >= 2.3 | ||
|
||
Windows | ||
~~~~~~~ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix Python version inconsistency in Windows section The Windows section still references Python 3.8, which contradicts the new 3.9+ requirement. Apply this change: -You need an environment with Python version 3.8 or above.
+You need an environment with Python version 3.9 or above.
|
||
|
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.
💡 Codebase verification
Ubuntu 24.04 is not yet officially supported by Read the Docs
Ubuntu 24.04 (Jammy) was just released in April 2024 and Read the Docs typically takes some time to officially support new Ubuntu LTS versions. I recommend keeping ubuntu-22.04 for now to ensure stable documentation builds. You can upgrade to 24.04 once it's officially supported by Read the Docs.
.readthedocs.yml
back toubuntu-22.04
mambaforge-23.11
as it's independent of the OS version🔗 Analysis chain
Verify Ubuntu 24.04 support on Read the Docs.
While upgrading to Ubuntu 24.04 LTS is forward-thinking, we should verify that Read the Docs officially supports this version to ensure stable documentation builds.
Let me gather more information about Read the Docs' Ubuntu support.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
Length of output: 474
Script:
Length of output: 944