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

The reversed top/bottom values from IVRSystem::GetProjectionRaw could cause injury with an Odyssey+ headset #1050

Open
Scawen opened this issue Feb 25, 2019 · 0 comments

Comments

@Scawen
Copy link

Scawen commented Feb 25, 2019

Hello, I'm the programmer of Live for Speed. We have been investigating an issue for one of our customers who has a Samsung Odyssey+ headset. He has previously used an older Odyssey with no issues. LFS has always worked well with the Vive. The reported problem was that the right eye's image seemed to be up too high. It turns out the Odyssey+ has different Top and Bottom values for both eyes, and this can cause serious problems if you take the values from IVRSystem::GetProjectionRaw at face value.

The bug was already reported but still exists:
#110
#816

The report for issue 110 is a good description. The "Top" value actually refers to the bottom, while the "Bottom" value refers to the top.

It is particularly bad with the Odyssey+ because of the slightly different height of the left and right screens. I have no idea why they have different height screens, it seems a strange design. But because they are only slightly different, if this height difference is corrected the wrong way round, it's not just a distortion issue as you might expect. A user can get used to it then have vision problems when they remove the headset.

Here is a link to the thread on our forum where we have discussed the problem, done some logging and come to the result that agrees with the existing reports linked above:
https://www.lfs.net/forum/thread/93043

It look like I now need to do a new release of Live for Speed, using the reversed values.

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

No branches or pull requests

1 participant