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

Switch to a "static standing" universe. #192

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kitlith
Copy link
Member

@kitlith kitlith commented Feb 24, 2023

This is WIP and has yet to be tested. Would not be surprised if applying the translation and rotation is currently incorrect. Just haven't gotten around to testing/tweaking it yet.

Feel free to take this and run with it on your own, this should get the concept across.

This is WIP and has yet to be tested. Would be completely unsurprised if
applying the translation and rotation is currently incorrect. Just
haven't gotten around to testing/tweaking it yet.
@kitlith
Copy link
Member Author

kitlith commented Mar 2, 2023

Finally got around to testing this today, looks like I stuffed up in two respects:

  • There's a trivial typo to fix (and formatting)
  • Apparently export_live_to_buffer is returning None, which means I stuffed up in ovr_overlay >_>

Still need to figure out how to apply the universe translation/rotation.
In the driver, we just pass it as the 'WorldFromDriver'
translation/rotation.

Parsing the current universe works though, now, after merging the fixup
PR to ovr_overlay.
@TheButlah
Copy link
Collaborator

Anything I can do on my end to move this forward? Would love if the overlay can be fixed

@kitlith
Copy link
Member Author

kitlith commented Mar 13, 2023

Anything I can do on my end to move this forward?

We just need to figure out how to apply the universe offset here, which we didn't have to do in the driver because steamvr had a separate field for this. I'm pretty sure that's the only thing being done incorrectly right now.

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