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

{tools}[foss/2021b] OpenAI-Gym v0.21.0 w/ Python 3.9.6 #14779

Merged

Conversation

@lexming lexming added the update label Jan 19, 2022
@boegel boegel added this to the 4.x milestone Feb 6, 2022
@boegel
Copy link
Member

boegel commented Feb 7, 2022

Test report by @boegel
FAILED
Build succeeded for 0 out of 1 (1 easyconfigs in total)
node3119.skitty.os - Linux UNKNOWN UNKNOWN, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/67a5c88672cdfc3194533b335d3e40fa for a full test report.

@boegel
Copy link
Member

boegel commented Feb 7, 2022

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3523.doduo.os - Linux UNKNOWN UNKNOWN, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/d33d99f6019ebc7d4d65de8f42aeae1b for a full test report.

@easybuilders easybuilders deleted a comment from boegelbot Feb 7, 2022
@boegel
Copy link
Member

boegel commented Feb 7, 2022

@lexming Failed test report is with read-only install dir configuration, the import check tries to change something in the installation?!

@lexming
Copy link
Contributor Author

lexming commented Feb 7, 2022

soooo... mujoco-py tries to create a lock in the library directory of MuJoCo /facepalm

https://github.com/openai/mujoco-py/blob/f1312cceeeebbba17e78d5d77fbffa091eed9a3a/mujoco_py/builder.py#L89-L92

There are already open issues about it at openai/mujoco-py#424 and openai/mujoco-py#533

I'll make a patch to use a temp dir instead

@lexming
Copy link
Contributor Author

lexming commented Feb 9, 2022

@boegel should be fixed in #14965

@migueldiascosta
Copy link
Member

@boegelbot please test @ generoso

@boegelbot
Copy link
Collaborator

@migueldiascosta: Request for testing this PR well received on login1

PR test command 'EB_PR=14779 EB_ARGS= /opt/software/slurm/bin/sbatch --job-name test_PR_14779 --ntasks=4 ~/boegelbot/eb_from_pr_upload_generoso.sh' executed!

  • exit code: 0
  • output:
Submitted batch job 8112

Test results coming soon (I hope)...

- notification for comment with ID 1034535489 processed

Message to humans: this is just bookkeeping information for me,
it is of no use to you (unless you think I have a bug, which I don't).

@boegelbot
Copy link
Collaborator

Test report by @boegelbot
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
cns1 - Linux Rocky Linux 8.5, x86_64, Intel(R) Xeon(R) CPU E5-2667 v3 @ 3.20GHz (haswell), Python 3.6.8
See https://gist.github.com/96a73fe940b569bf0e2c39b75c70f6fc for a full test report.

@boegel
Copy link
Member

boegel commented Feb 28, 2022

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3120.skitty.os - Linux CentOS Linux 7.9.2009, x86_64, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 3.6.8
See https://gist.github.com/a9c7528375956f853b7557c407e37a6a for a full test report.

Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel
Copy link
Member

boegel commented Feb 28, 2022

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in total)
node3623.doduo.os - Linux RHEL 8.4, x86_64, AMD EPYC 7552 48-Core Processor (zen2), Python 3.6.8
See https://gist.github.com/81c303c8e89231ffaaf5d3d82fd8dd7a for a full test report.

@boegel
Copy link
Member

boegel commented Feb 28, 2022

Going in, thanks @lexming!

@boegel boegel merged commit 4ccb441 into easybuilders:develop Feb 28, 2022
@lexming lexming deleted the 20220119143822_new_pr_OpenAI-Gym0210 branch February 28, 2022 17:10
@boegel boegel modified the milestones: 4.x, next release (4.5.4?) Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants