-
Notifications
You must be signed in to change notification settings - Fork 695
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Please specify in readme.md that is for Python3.8-3.9. 3.8+ is misleading because 3.10 leads to problems #175
Comments
Hey @ElliotMunro200 thanks for reporting. Would you have the bandwidth to submit a PR to fix this? If not no worries I can do it. |
I'm very new with Git and Github but I've forked the project and created a PR. Hope it works fine. Let me know if I need to change the update. Cheers. |
@ElliotMunro200 can you tell what problems you had with python >= 3.10 ? i would like to analyze the issue |
If I remember correctly, this is because some dependencies do not support 3.10.. |
I can't remember what I faced at the time when I originally did this.
I just made a fresh conda env with Python 3.10 and tried to install cleanrl
with "pip install -r requirements/requirements.txt".
I got many errors like "Ignoring absl-py: markers 'python_full_version >=
"3.7.1" and python_version < "3.10"' don't match your environment".
Neither could I execute "poetry install" as I got the error "Current Python
version (3.10.0) is not allowed by the project (>=3.7.1,<3.10).
Please change python executable via the "env use" command."
Hope this helps.
Cheers,
Elliot.
…On Wed, 3 May 2023 at 00:55, Costa Huang ***@***.***> wrote:
@ElliotMunro200 <https://github.com/ElliotMunro200> can you tell what
problems you had with python >= 3.10 ? i would like to analyze the issue
If I remember correctly, this is because some dependencies do not support
3.10..
—
Reply to this email directly, view it on GitHub
<#175 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ARXWDUGURTKNIG437HPQE23XED72RANCNFSM5UOJQEDQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Problem Description
Current Behavior
Expected Behavior
Possible Solution
Steps to Reproduce
The text was updated successfully, but these errors were encountered: