Usage of Camera.main throughout the repository

Using Camera.main basically, does a FindByTag on the scene and searches through all tagged objects to find the main camera. With more tagged objects the performance gets worse and worse. Instead, the reference should be cached in Awake/Start. There are a lot of occurrences of Camera.main in the repository that doesn’t do this and might potentially cause performance issues at some point or at least drain the performance unnecessarily. Some scripts call Camera.main up to 8 times on every Update. There are functions that will have to be made obsolete like MathUtils.IsInFOV(position) to include the camera reference in the call.

[Zod-]

I don’t think the GazeManager would be a good place for it since it would require an instance of the GazeManager and make the utility scripts coupled to it. This should probably be standalone.

[Zod-]

My first thought was that a project that changes the main camera often would probably implement their own system to handle this. But then I realized they would have to update the references in every single script, which will probably be quite a challenge to keep track of all of them in the scene and all the different ways they are cached (GazeTransform, TargetTransform, mainCamera, MainCamera). Some scripts like HandDraggable even have private caches so this wouldn’t be entirely possible.

I’m not sure what the best way would be to handle this.

If we just keep the references public, add an interface to at-least make this consistent? Some scripts cache the transform and some the camera.

Provide a cache version of the camera through an extension like for example Camera.cachedMain?