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.

Sign up for GitHub

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

Jump to bottom

Consolidation #12

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

Consolidation #12

wants to merge 27 commits into from

Conversation

kylemcdonald
Copy link

This is tested with OF 0.9.0 for OSX.

  • Merges work from @mattebb to master.
  • Adds a fix for using background color when it is set (instead of defaulting to blue).
  • Actually turns predictive pose on and off (I think. I can't really tell.)

Edit: I take it back, still working on this. The example doesn't run anymore, but my code does...

mattebb and others added 27 commits October 13, 2014 23:44
@mattebb
Several fixes, added positional tracking
1a71a43 
I’ve made a copy of the example scene, which I can merge back in later
when this is all clean. My intention for my own projects is to use the
GL3 programmable renderer pipeline, so will work on making that usable
too.

There were a few confounding factors in getting this working:

* The scene scale in the included example was huge, at non-physical
size (500 m wide). This meant that even when I did get positional
tracking was working, I didn’t notice it, since any head movements
didn’t give a sense of parallax with distant objects. It also had the
side effect of making any stereo effects un-noticeable since the eye
separation offsets coming from the had are also in physical scale.

* Part of the culprit here is the ofEasyCam which by default translates
the camera quite a large distance away from the origin (based on
viewport size).  This is now disabled with cam.setAutoDistance(false);

* The previous example also had drawing errors due to lack of depth
testing in the Fbo. This is now enabled.
@mattebb
WIP getting programmableRenderer working
9136abf 
Still a big mess, but getting closer
@mattebb
Working properly in programmableRenderer
85a671a 
and default fixed function too
@mattebb
Cleanup, need cam.begin() and end()
2ff12f5 
Camera worked ok in non-programmable renderer, but needs begin() and
end() around drawing, since programmableRenderer doesn’t seem to set up
the viewport the same way…
@mattebb
More fixes
f11364f 
Scratch last commit, had things very backwards
@mattebb
Updated to SDK v0.43
a00a17d 
May need to re-add some files for windows
Also need to update deprecated functions
@mattebb
Replace deprecated ovrHmd_GetEyePosePerEye
7516700 
Now uses ovrHmd_GetEyePoses
@mattebb
Cam works, doesn't work, works, doesn't work, now works...
6e1fbbb
@mattebb
For testing - get rid of viewAdjust
3f570c6 
Taken care of by getEyePoses now, apparently
@mattebb
WIP code for SDK distortion rendering
75f8f62 
Not working yet
@mattebb
Getting closer to SDK render working
45ed51a 
still weird issues with render target vs. window size
@mattebb
#ifdefs now for SDK rendering, fixed final render target size
910bfe6
@mattebb
Duh, matrix order
0f43c1e
@mattebb
Set low persistence on for SDK render
1fb51b7
@mattebb
if, not ifdef
6b318bd
@mattebb
SDK Render working in fixed function pipe now
40153ab 
All it needed was glUseProgram(0);

sigh…
@mattebb
Fix shaders for client distortion
6ca0291
@mattebb
Cleanups, dismiss safety warning
4986e42
@mattebb
clean up frag shader
8a50719
Update project files
43464d3
Roll in patch from armadillu
c9e78b9
Initialise hmdCaps properly
c9d6945
@obviousjim
adding orientation mat back
8194d78
@kylemcdonald
background from ofGetStyle and predictive orientation put to use
81db5e3
@kylemcdonald
merged mattebb
e46aad2
@kylemcdonald
removed example that was said to 'have problems'
28a13ee
@kylemcdonald
removed couts
3c835a3
@obviousjim
Copy link
Owner

wow!

will review on Mac/Win and integrate if no issues are found

On Wed, Mar 11, 2015 at 3:31 AM, Kyle McDonald [email protected]
wrote:

This is tested with OF 0.9.0 for OSX.

  • Merges work from @mattebb https://github.com/mattebb to master.
  • Adds a fix for using background color when it is set (instead of
    defaulting to blue).
  • Actually turns predictive pose on and off (I think. I can't really
    tell.)

You can view, comment on, or merge this pull request online at:

#12
Commit Summary

  • Several fixes, added positional tracking
  • WIP getting programmableRenderer working
  • Working properly in programmableRenderer
  • Cleanup, need cam.begin() and end()
  • More fixes
  • Updated to SDK v0.43
  • Replace deprecated ovrHmd_GetEyePosePerEye
  • Cam works, doesn't work, works, doesn't work, now works...
  • For testing - get rid of viewAdjust
  • WIP code for SDK distortion rendering
  • Getting closer to SDK render working
  • #ifdefs now for SDK rendering, fixed final render target size
  • Duh, matrix order
  • Set low persistence on for SDK render
  • if, not ifdef
  • SDK Render working in fixed function pipe now
  • Fix shaders for client distortion
  • Cleanups, dismiss safety warning
  • clean up frag shader
  • Update project files
  • Roll in patch from armadillu
  • Initialise hmdCaps properly
  • adding orientation mat back
  • background from ofGetStyle and predictive orientation put to use
  • merged mattebb
  • removed example that was said to 'have problems'

File Changes

Patch Links:

—
Reply to this email directly or view it on GitHub
#12.

  • James

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kylemcdonald @obviousjim @mattebb