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

Tracking rewrite #2481

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Tracking rewrite #2481

wants to merge 1 commit into from

Conversation

zmerp
Copy link
Member

@zmerp zmerp commented Oct 28, 2024

The goal is to move prediction on the server side for Monado support and implement better tracking algorithms

@zmerp
Copy link
Member Author

zmerp commented Oct 28, 2024

@The-personified-devil if you want you can continue the discussion here

@The-personified-devil
Copy link
Collaborator

My point about the offset is that if we're already lying to Steamvr by predicting ahead of the frame render time we should at least tell it the most accurate info. I.e. that there's no additional offset to correct for. The only question is how to implement prediction increase. We can increase the offset to make steamvr predict more, or add or subtract from the predict to timestamp (or perhaps even adjust algos). Or we make a second setting for it

@zmerp
Copy link
Member Author

zmerp commented Oct 31, 2024

My point about the offset is that if we're already lying to Steamvr by predicting ahead of the frame render time we should at least tell it the most accurate info. I.e. that there's no additional offset to correct for

The thing is we want the final rendered frame to have the pose that the client expects. We have to trick SteamVR by partially predicting the poses in advance, so that when it predicts to its virtual vsync, it's actually predicting to the real vsync.

We can increase the offset to make steamvr predict more

We tried, and this upsets SteamVR, causing spikes in game render latency or just clamping the prediction.

Or we make a second setting for it

We can try this

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