pex-renderer icon indicating copy to clipboard operation
pex-renderer copied to clipboard

Orbiter upgrade

Open vorg opened this issue 6 years ago • 2 comments

  • Pinch to zoom
  • Allow longitude < 0 and > 360 to avoid rotation interpolation bugs (this is potentially breaking)
  • Code refactoring

vorg avatar Feb 14 '19 16:02 vorg

Should it be the same logic for this.currentLat ?

dmnsgn avatar Feb 14 '19 16:02 dmnsgn

I've just realized that having lon going > 360'deg while fixes rotating backwards for CX project it might be problematic when working on a project involving globe. E.g.

You spin few times arriving at lon: 1240deg and then you click location on a globe and set lon programmatically orbiter.set({ lat: 51, lon: 0.1 }). That would cause orbiter to linearly interpolate from 1240 -> 0.1 and therefore spin 3.5x.

I propose we ship custom orbiter for CX project as it is here now and thing a bit more on the proper solution for pex-renderer.

vorg avatar Feb 15 '19 02:02 vorg

Closing as refactored orbiter is now in pex-cam.

dmnsgn avatar May 20 '24 09:05 dmnsgn