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

Opengl3 example #185

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ChristophHaag
Copy link
Contributor

Not perfect but good enough to serve as an example now.

The bad:

  • Somewhat inefficient, uploads one model matrix for each cube every frame, and even when cubes are static (should run well on pretty much all hardware, especially VR capable)
  • autotools build missing
  • not sure if all hotkeys still work
  • math_3d.h is from an external project (MIT license), can be replaced later
  • HMD origin is on the floor - HMDs without positional tracking are stuck on the floor.

Move the floor a bit down? Then HMDs with positional tracking are floating too high. Make HMDs without positional tracking report a constant y pos of about 1.8meter?

The good:

  • It works
  • It demonstrates how to render into a texture and then in a second pass render the texture with distortion correction to a window with gl3
  • It renders blocks representing the controller position (TODO: dummy controllers always in (0,0,0))

- math_3d.h may be replaced by internal openhmd functionality

v2: use one triangle instead of quad and fix some comments
    also comment out gl debug
v3: update to openhmd master and fix some code issues
@TheOnlyJoey TheOnlyJoey self-requested a review February 4, 2020 16:05
@ChristophHaag ChristophHaag force-pushed the opengl3-example branch 4 times, most recently from 02cd1a1 to 4427f79 Compare February 4, 2020 17:54
@TheOnlyJoey
Copy link
Member

Just needs better defaults for object location/orientation and look into the new api call for controller vector.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants