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

Updated HDK Server file to disable server side prediction #469

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

Updated HDK Server file to disable server side prediction #469

wants to merge 1 commit into from

Conversation

demonixis
Copy link

This PR is related to the issue #441

@rpavlik
Copy link
Member

rpavlik commented Sep 9, 2016

Thanks for this - I asked Russ for a clarification because I was under the impression that while yes, we technically are doing client side prediction in extended mode, we don't actually have the frame timing data to do it correctly there, and so in my eventual future plans of doing this one myself I was going to turn off (or perhaps just reduce to a single-eye on 1.x) client side prediction in the extended mode RM configs.

@DuFF14
Copy link
Member

DuFF14 commented Sep 10, 2016

I haven't compared server-side vs extended client-side , but I will put it on my list.

Does the fusion plugin handle server-side prediction? Will removing this section impair positional tracking accuracy?

"aliases": {
        "/headSpace": {
            "translate": [0.0, 0.0, 0.04141],
            "child": "/org_osvr_filter_videoimufusion/HeadFusion/semantic/fused"
        },
        "/me/head": "/headSpace"
    }

@russell-taylor
Copy link
Contributor

As I commented on Ryan's question on the issue, I think I set the extended mode prediction to be half a frame longer than the direct-mode prediction so that we're using the expected delay. Of course, actual delay will depend on application rendering time. The client-side timing info returns 0 in the case of no information, so kind of auto-disables itself in that case.

@demonixis
Copy link
Author

Hi, any news about this PR? Performances are critical for new users and the current configuration is not adapted.

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.

4 participants