-
Notifications
You must be signed in to change notification settings - Fork 724
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
[Bug Report] WebRTC streaming does not work on Orbit scripts with Isaac Sim 2023.1.1 #268
Closed
2 tasks done
Comments
ADebor
pushed a commit
to ADebor/IsaacLab
that referenced
this issue
Apr 8, 2024
# Description The Nucleus directory for materials has changed in 2023.1. They are now directly inside `NVIDIA/Materials` instead of `NVIDIA/Assets/Materials`. This MR fixes the referenced directory. ## Type of change - Bug fix (non-breaking change which fixes an issue) ## Screenshots This is how the rough terrain looks like now: ![shingle_ground](https://github.com/isaac-orbit/orbit/assets/12863862/c561f664-22bb-45d1-a713-9d95ec48feda) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./orbit.sh --format` - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
Update: it seems like the problem is not specific to Orbit, as the livestream.py example of Isaac Sim also fails the same way when WebRTC is enabled. I wonder if there is a workaround we could implement in Orbit. |
3 tasks
Closing this issue to continue the discussion in #381. |
Mayankm96
added
duplicate
This issue or pull request already exists
invalid
This doesn't seem right
labels
Apr 26, 2024
fatimaanes
pushed a commit
to fatimaanes/omniperf
that referenced
this issue
Aug 8, 2024
# Description The Nucleus directory for materials has changed in 2023.1. They are now directly inside `NVIDIA/Materials` instead of `NVIDIA/Assets/Materials`. This MR fixes the referenced directory. ## Type of change - Bug fix (non-breaking change which fixes an issue) ## Screenshots This is how the rough terrain looks like now: ![shingle_ground](https://github.com/isaac-orbit/orbit/assets/12863862/c561f664-22bb-45d1-a713-9d95ec48feda) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./orbit.sh --format` - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
If I use the 2023.1.1 version of Isaac Sim as a base in Orbit and enable WebRTC livestream on Orbit scripts, WebRTC does not start correctly, and I get the following errors on the terminal:
Steps to reproduce
Run random_ageny.py with WebRTC enabled (livestream=3) on any environment.
System Info
Additional context
WebRTC works correctly with Orbit scripts if the Isaac Sim version is 2023.1.0-hotfix.1. The problem is specific to the 2023.1.1 version.
Interestingly, even on version 2023.1.1, running directly
runheadless.webrtc.sh
from the container successfully enables the WebRTC. Hence, the problem is most probably related to Orbit.Checklist
Acceptance Criteria
The text was updated successfully, but these errors were encountered: