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

Example of creating a stereo agent #62

Merged
merged 12 commits into from
Jun 5, 2019
Merged

Conversation

erikwijmans
Copy link
Contributor

Motivation and Context

See #56 for context!

This one adds an example of creating an agent that uses a stereo RGB pair or a stereo depth pair. The displaying to the user is done via cv2

How Has This Been Tested

depth_example

rgb_example

Also via the new smoke test in pytest.

Types of changes

  • New feature (non-breaking change which adds functionality)

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label May 31, 2019
Copy link
Collaborator

@msavva msavva left a comment

Choose a reason for hiding this comment

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

This is awesome -- thanks @erikwijmans . Left a few comments on typos. Also, color frames look like they have BGR vs RGB color channel flip (likely cv2 expects the former channel order by default). Other than these small fixes, LGTM! 👍

examples/stereo_agent.py Outdated Show resolved Hide resolved
examples/stereo_agent.py Outdated Show resolved Hide resolved
examples/stereo_agent.py Outdated Show resolved Hide resolved
examples/stereo_agent.py Outdated Show resolved Hide resolved
examples/stereo_agent.py Outdated Show resolved Hide resolved
@erikwijmans
Copy link
Contributor Author

BGR vs RGB color channel flip

I am so bad at remembering to do that.....

Copy link
Contributor

@mathfac mathfac left a comment

Choose a reason for hiding this comment

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

Looks awesome to see stereo pair in HSIM, some minor comments.

examples/stereo_agent.py Outdated Show resolved Hide resolved
examples/stereo_agent.py Show resolved Hide resolved
examples/stereo_agent.py Show resolved Hide resolved
@msavva
Copy link
Collaborator

msavva commented Jun 5, 2019

@erikwijmans since this PR has now been reviewed, shall we land it for release?

@erikwijmans erikwijmans merged commit e8e14b9 into master Jun 5, 2019
@erikwijmans erikwijmans deleted the stereo-agent-example branch June 5, 2019 18:09
Ram81 pushed a commit to Ram81/habitat-web-sim that referenced this pull request Dec 10, 2020
* Stereo agent example

* Add test

* Add some comments

* Render at 256

* Smoke test more

* Mark as gfx

* Typos, BGR vs RGB, and get ready for multiple examples to test!

* Little change

* Rename

* Nits
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants