Skip to content

[BUG FIX] In go2_backflip.py, env.reset should return one value not two.#2986

Open
kshitijgoel007 wants to merge 1 commit into
Genesis-Embodied-AI:mainfrom
kshitijgoel007:bugfix/examples/go2_backflip
Open

[BUG FIX] In go2_backflip.py, env.reset should return one value not two.#2986
kshitijgoel007 wants to merge 1 commit into
Genesis-Embodied-AI:mainfrom
kshitijgoel007:bugfix/examples/go2_backflip

Conversation

@kshitijgoel007

Copy link
Copy Markdown

Description

The change here 0439608#diff-c9e3853aba220342efbefc93fb41b9ef5096a6ef189f8e2f973642779c813c97L247 has introduced a bug in the go2_backflip example where 2 values are expected out of the env.reset call.

Related Issue

Resolves #2985

Motivation and Context

Examples should run without crashing.

How Has This Been / Can This Be Tested?

I ran the example using python examples/locomotion/go2_backflip.py

Screenshots (if appropriate):

Checklist:

  • I read the CONTRIBUTING document.
  • I followed the Submitting Code Changes section of CONTRIBUTING document.
  • I tagged the title correctly (including BUG FIX/FEATURE/MISC/BREAKING)
  • I updated the documentation accordingly or no change is needed.
  • I tested my changes and added instructions on how to test it for reviewers.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

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.

[Bug]: go2_backflip fails at env.reset because it is expecting two return values from go2_env

1 participant