threejs r89 not support? about three.vrcontroller HOT 13 OPEN

A forked branch fixes somethings that have been merged and causing problems with Daydream / Gear.

Here is the differences. It still has issues with height data when standing up. The entire floor and controller moves when standing up. It could be a problem still with VRController not sure.

https://jsfiddle.net/danrossi303/13onbkt0/
https://dev.electroteque.org/vrcontroller/raycast.html

fixed branch.

https://jsfiddle.net/danrossi303/orfktz0h/1/
https://dev.electroteque.org/vrcontroller/raycast2.html

from three.vrcontroller.

I have made fixes in my version for r91. I tried to get through to them regards to problem with DayDream based controllers but unfortunately the standing changes they made are set.

A work around is required by not setting a standing matrix for DayDream / GearVR.

My version supports GearVR also. Will update tests eventially.

here is where the fix needs to be made.

https://github.com/danrossi/THREE.VRController/blob/master/VRController.js#L484

from three.vrcontroller.

So r89 came out ... 2 days ago? ;)

If there is a compatibility issue with r89 can you take that on? (What are you seeing in the console? Was there a Three.js API update that broke VRController? If so, is it as simple as changing a property name to match the new API?) I haven’t had a chance to look at r89 yet but you say this happened when you upgraded—so everything was working fine for you in r88 or r87?

Also, can you please note the browser and VR rig you’re using? Is this for mobile? Desktop?

from three.vrcontroller.

r88 and r87 are work, but r89 has a error in Windows 10 and Firefox nightly:

from three.vrcontroller.

This looks like it is due to a fundamental change in how Three.js thinks about a VR headset’s standingMatrix. Here is the discussion about Three’s use of standing matrices in WebVR: mrdoob/three.js#11619

And here is @mrdoob’s solution in practice: mrdoob/three.js@3cb51d8

I’m currently working on a patch for VRController and the demo example so it can support Three.js r89 and above—but feel free to beat me to it with a pull request. I’m also migrating Space Rocks to r89 so that can serve as a more fleshed out example.

from three.vrcontroller.

Thanks a lot！

from three.vrcontroller.

It's implemented here but a possible bug. The controllers are too low now. Everything sits on the floor too I think

mrdoob/three.js#13158

from three.vrcontroller.

This patch in the PR seems to make the controller above your head when standing up.

from three.vrcontroller.

Hi the standingmatrix code has been merged. However the original PR before changes I confirmed was working with VRController. Now I can't see the controller object unless I comment out the standingmatrix code for Daydream view.

See the test links here.

mrdoob/three.js#13158 (comment)

I have made an improved demo with an updating laser line and marker pointer also. I have also provided support for GearVR in my version. Should I do a PR for this ?

https://github.com/danrossi/THREE.VRController

from three.vrcontroller.

According to GearVR the controller is above the head. Possibly doubling of the matrix ?

from three.vrcontroller.

example of all fixes in that branch and current three.js is here

http://dev.electroteque.org/vrcontroller/raycast.html

from three.vrcontroller.

An example implementation with a laser pointer and fallback gaze control is here.

https://github.com/danrossi/three-vr-controls/blob/master/src/VRControls.js

from three.vrcontroller.

Another example with fixes; this one works with THREE r95 on the Oculus Go:
https://github.com/bhautikj/THREE.CommonClicker

Short summary of fixes:

• Based on: https://github.com/stewdio/THREE.VRController/pull/20/files?utf8=%E2%9C%93&diff=unified&w=1
• Add a function to check if it's the Oculus Go browser:

var CheckIfOculusGo = function() {
  return navigator.userAgent.includes('OculusBrowser');
}

• On renderer initialization:

if (!CheckIfOculusGo()) {
  renderer.vr.standing = true
}
renderer.vr.enabled  = true;
// use up-to-date button for WebVR
document.body.appendChild( WEBVR.createButton( renderer ) );

• In the function thats triggered on the vr controller connected event:

window.addEventListener( 'vr controller connected', function( event ){
...
  if (!CheckIfOculusGo()){
    controller.standingMatrix = renderer.vr.getStandingMatrix()
  } else {
    if ( !controller.armModel ) controller.standingMatrix = renderer.vr.getStandingMatrix();
  }
...

from three.vrcontroller.