You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
PolylinePipeline.wrapLongitude currently takes an ellipsoid and array of positions, but the positions are always assumed to be in earth-fixed. If you create a Polyline collection and assign the modelMatrix property (for example to draw inertial orbit paths), we incorrectly pass non earth-fixed positions to PolylinePipeline.wrapLongitude, causing incorrect results and horizontal lines that run across the entire map. The simple (but slow) fix is to have Polyline translate each of its positions to earth-fixed using modelMatrix before calling PolylinePipeline.wrapLongitude. Since this would probably have a large impact on performance, I hope we can figure out a better way to do it.
This may not even be that big of a deal for the time being, since we will most likely always draw earth-fixed paths in 2D and Columbus view, but ultimately we want that to be a user option.
The text was updated successfully, but these errors were encountered:
PolylinePipeline.wrapLongitude
currently takes an ellipsoid and array of positions, but the positions are always assumed to be in earth-fixed. If you create a Polyline collection and assign themodelMatrix
property (for example to draw inertial orbit paths), we incorrectly pass non earth-fixed positions toPolylinePipeline.wrapLongitude
, causing incorrect results and horizontal lines that run across the entire map. The simple (but slow) fix is to havePolyline
translate each of its positions to earth-fixed usingmodelMatrix
before callingPolylinePipeline.wrapLongitude
. Since this would probably have a large impact on performance, I hope we can figure out a better way to do it.This may not even be that big of a deal for the time being, since we will most likely always draw earth-fixed paths in 2D and Columbus view, but ultimately we want that to be a user option.
The text was updated successfully, but these errors were encountered: