compatability with react-three-fiber? about camera-controls

[ammanvedi]

just got this working here is my final code, seems to be working nicely thank you to both above i was trying to get a nice solution for dollyToCursor with OrbitControls with no luck and this one is perfect, i can go to bed now XD

import React from 'react';
import * as THREE from 'three';
import { extend, ReactThreeFiber, useFrame, useThree } from '@react-three/fiber';
import CameraControls from 'camera-controls';

CameraControls.install({ THREE: THREE });

extend({ CameraControls });

declare global {
    namespace JSX {
        interface IntrinsicElements {
            cameraControls: ReactThreeFiber.Object3DNode<
                CameraControls,
                typeof CameraControls
            >;
        }
    }
}

export const CustomOrbitControls: React.FC<any> = React.forwardRef((props, ref) => {
    const { camera, gl } = useThree();
    useFrame((state, delta) => {
        if (ref.current) {
            ref.current.update(delta);
        }
    });
    return (
        <cameraControls
            ref={ref}
            args={[camera, gl.domElement]}
            dollyToCursor
            mouseButtons={{
                left: CameraControls.ACTION.TRUCK,
                middle: CameraControls.ACTION.NONE,
                right: CameraControls.ACTION.NONE,
                shiftLeft: CameraControls.ACTION.NONE,
                wheel: CameraControls.ACTION.ZOOM,
            }}
        />
    );
});

[yomotsu]

Seems delta of every frame in your app is too small. delta is supposed to be 0.01666... sec or so.

You can use getElapsedTime() to get correct delta, rather than getDelta().

const CControls: React.FC<any> = () => {
  const ccontrolRef = useRef<any>(null);
  const { camera, gl, clock, scene } = useThree();
  let lastTime = 0;
  useFrame(() => {
    if (ccontrolRef.current) {
      const elapsed = clock.getElapsedTime();
      const delta = elapsed - lastTime;
      console.log( delta );
      //   ccontrolRef.current.update();
      const hasControlsUpdated = ccontrolRef.current.update(delta);
      if (hasControlsUpdated) gl.render(scene, camera);

      lastTime = elapsed;
    }
  });
  return <cameraControls ref={ccontrolRef} args={[camera, gl.domElement]} />;
};

If you call getDelta() somewhere else, the delta time will be the diff between "somewhere" and the useFrame.
I think this is not a problem of camera-controls.

[kurosh-z]

I don't know about internals of react-three-fiber but I don't use getDelta() in my app yet!
anyway that definitely solved the problem!
thanks again for investigating the issue.

[jfabraxas]

FYI: Second arg of useFrame callback is delta:

 useFrame((state, delta) => {
    if (ccontrolRef.current) {
      const hasControlsUpdated = ccontrolRef.current.update(delta);
      if (hasControlsUpdated) gl.render(scene, camera);
    }
  });

rf3 doc

[yomotsu]

Did you try without camera controls?
Did you also try with threejs official orbit controls?

Then if still it happens, can you provide a simplified working example on jsfiddle or somewhere? Otherwise I need to prepare the environment to reproduce.

[kurosh-z]

Thank you for the speedy reply!
yes I have used OrbitControls and it works fine!
here is the codesanbox version with some simplifications. you can click on the button to switch the controllers.
https://codesandbox.io/s/blissful-chandrasekhar-ew99r

[yaroslavnikiforov]

You don't need this line:
if (hasControlsUpdated) gl.render(scene, camera);