-
Notifications
You must be signed in to change notification settings - Fork 260
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
Update mujoco-py to mujoco and gym to gymnasium + more #421
Update mujoco-py to mujoco and gym to gymnasium + more #421
Conversation
…ta-World consistency in README & CONTRIBUTING
Migrate the door envs to the new MuJoCo bindings & a couple minor changes
Reset from initial qpos config
Fixed plate-slide-v2
…y as far as RNG seeding goes
Migrate handle- envs to new MuJoCo bindings + fix misc issues (hand pos / rng)
Merging updated fixing issues from Farama-Foundation branch
The CI is crashing currently due to too much process - My guess is that all of the environments are initialised using memory even when not used however we do something similar in gymnasium with no issue (on a smaller scale admittedly) |
Could you summarise the changes in the PR description |
Tested scripted policies
Fixing issues
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.
Due to the number of interlinked changes, we have decided not to split this into several smaller pr (going forward this is the plan however).
Our plan is to continue improving the implement with this PR not being the final changes
README.md
Outdated
@@ -20,23 +20,25 @@ __Table of Contents__ | |||
- [Acknowledgements](#acknowledgements) | |||
|
|||
## Join the Community | |||
<<<<<<< HEAD |
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.
Remove
metaworld/__init__.py
Outdated
env_cls = classes[env_name] | ||
env = env_cls() | ||
for (env_name, args) in args_kwargs.items(): | ||
print(env_name, args) |
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.
Remove
@@ -24,6 +24,6 @@ | |||
<position ctrllimited="true" ctrlrange="-1 1" joint="l_close" kp="400" user="1"/> | |||
</actuator> | |||
<equality> | |||
<weld body1="mocap" body2="hand" solref="0.02 1"/> | |||
<weld body1="mocap" body2="hand" solref="0.02 1" /> |
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.
Was this intentional?
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.
@pseudo-rnd-thoughts nope
19411bd
into
Farama-Foundation:master
This PR updates multiple things: