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

Load Scene button is disabled even with valid Mesh/PLY filepaths in OrbitNavImporter extension #1

Closed
dillonloh opened this issue Oct 20, 2024 · 2 comments

Comments

@dillonloh
Copy link

Summary

The Load Scene button in the UI for Orbit Nav Importer extension is grayed out and unable to be used. This happens even if I load valid filepaths for the Mesh and PLY fields. However, Reset Scene button works fine.

Steps to reproduce

Environment: IsaacSim v4.2.0
Running on dev/add_sampling branch of this repo.
Install orbit (IsaacLab) v0.3.0 as per README and follow all steps to setup

Open IsaacSim, enable the Orbit Nav Importer extension, and attempt to use the UI. No matter how I changed the filepaths and toggled with the options, the Load Scene button remained grey.

Possible Source of Bug

In the file extensions/orbit.nav.importer/orbit/nav/importer/scripts/importer_ext.py, lines 345 and 349 have defined the load_btn input field to be disabled (refer to ss).

If i change it to True, the load button works fine, and the rest of the functionality involved in loading in the Matterport environment works with no issue.

image

What is the current bug behavior?

Load Scene button is disabled

What is the expected correct behavior?

Load Scene button should be enabled (maybe with conditions like valid filepaths provided etc.)

Relevant logs and/or screenshots

image

@dillonloh
Copy link
Author

just a tiny issue, not sure if its intended behaviour but it was blocking usage on my side so i just wanted to bring it up.
thanks for creating this project!

@pascal-roth
Copy link
Owner

Hi @dillonloh,

thanks a lot for using our work. The repo here is not really maintained anymore, please use leggedrobotics/viplanner#32 (or the main branch as soon as the PR is merged)

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

No branches or pull requests

2 participants