Skip to content

Added method to update window's title #20

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

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

rpapallas
Copy link
Contributor

I needed this small functionality and thought might be useful in general. I have multiple MuJoCo data that run in parallel and since I can only visualize one at a time, if I want to know which one I am looking at, I might need to update the window title.

@rohanpsingh
Copy link
Owner

rohanpsingh commented Aug 24, 2022

I guess it doesn't hurt anyone so I can merge this.

since I can only visualize one at a time

Could you please elaborate what you mean by this?

@rpapallas
Copy link
Contributor Author

rpapallas commented Aug 24, 2022

I have had issues spinning up multiple viewer instances from the same code. On macOS, it gives an error. I haven't tried on Linux, I could try now and see if this is a macOS issue? Are you able to spin up multiple instances?

Edit: I see, on Linux I could have two viewers, and it won't throw an error like macOS does, but only one viewer can render, the other one is black (even with different mjData).

@rohanpsingh
Copy link
Owner

Hi @rpapallas
I think in that case, I must try and make mutiple viewer instances work. Then, we can see if the method implemented in this PR is needed or not.

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.

2 participants