Webxr: Consider renaming `VRDisplay#getEyeParameters(VREye)` to `VRDisplay#eyes` array

Created on 28 Jan 2017  Â·  2Comments  Â·  Source: immersive-web/webxr

See OSVR's hmd.eyes array. Proposed sample usage:

vrDisplay.eyes;  // [<VR Eye offset="0.5" renderWidth="…" renderHeight="…">, <VR Eye offset="0.5" renderWidth="…" renderHeight="…">]

Most helpful comment

It's been my intention to get rid of VREyeParameters entirely, since it's mostly filled with values that could either be misused or should be delivered another way.

  • offset is baked into the view matrices
  • fieldOfView is baked into the projection matrices
  • canvasWidth/Height should be more flexible, allowing devs to choose quality over speed.

All 2 comments

It's been my intention to get rid of VREyeParameters entirely, since it's mostly filled with values that could either be misused or should be delivered another way.

  • offset is baked into the view matrices
  • fieldOfView is baked into the projection matrices
  • canvasWidth/Height should be more flexible, allowing devs to choose quality over speed.

This is effectively resolved in the 2.0 proposal so I'm closing this issue.

Was this page helpful?
0 / 5 - 0 ratings