Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Graphic Path type inconsistencies. #333

Open
alberto-acevedo opened this issue Jun 16, 2017 · 3 comments
Open

Graphic Path type inconsistencies. #333

alberto-acevedo opened this issue Jun 16, 2017 · 3 comments

Comments

@alberto-acevedo
Copy link
Contributor

There 3 path types : Rhumb (constant bearing), great circle (geodesic or shortest distance), and linear.

Cesium and WorldWind are defaulting path type to great circle. The SEC renderer is using rhumb. I checked and the EMP core geo libraries used for calculations are using great circle. So the only component using rhumb is the SEC renderer. There must be an option in the core and API to set the path type for all engines. What path types should be the default?

@alberto-acevedo
Copy link
Contributor Author

After further discussion and investigation the default path type should be rhumb for all engines. If rhumb is the path type of choice then there is no need to expose a property for path type in the EMP API.

Cesium will add an option for selecting rhumb type : CesiumGS/cesium#4000.

@alberto-acevedo
Copy link
Contributor Author

-WW Android is fine handling rhumb

  • Need to set rhumb paths in Leaflet and webWorldwWnd. I don't know is the engine's sdk support the use of rhumb.
    -issue Blocked until Cesium add support for rhumb path.

@alberto-acevedo alberto-acevedo removed this from the Sprint 2.8.0 milestone Jun 29, 2017
@alberto-acevedo
Copy link
Contributor Author

WSEC complaint that Android map calculations for graphics is incorrect because they are expecting geodesic and the map is rendering rhumb. The same issue happens also in emp3-web because it is using the same geo libraries Android map uses. The solution is to replace to library that supports geodesic like in the following link:

https://geographiclib.sourceforge.io/scripts/geod-calc.html

Cesium by default does support geodesic.
WW web ?
Leaflet ?

The Javascript renderer only supports rhumb. I need to talk to @michael-spinelli about the level of effort required to upgrade the renderer to use geodesic.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants