Merge branch 'master' of github.com:NASA-AMMOS/3DTilesRendererJS

Author: gkjohnson

CHANGELOG.md

@@ -8,6 +8,9 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Changed
 - EnvironmentControls: PivotMesh is now added to the scene before the "start" event is fired.
 
+### Fixed
+- Ellipsoid.getPositionToCartographic: fixed incorrect calculation of cartographic point.
+
 ## [0.4.23] - 2026-03-18
 ### Added
 - DebugTilesPlugin: Added support for "boundsColorMode".

src/three/renderer/math/Ellipsoid.js

@@ -353,7 +353,7 @@ export class Ellipsoid {
 		// From Cesium function Ellipsoid.cartesianToCartographic
 		// https://github.com/CesiumGS/cesium/blob/665ec32e813d5d6fe906ec3e87187f6c38ed5e49/packages/engine/Source/core/renderer/Ellipsoid.js#L463
 		this.getPositionToSurfacePoint( pos, _vec );
-		this.getPositionToNormal( pos, _norm );
+		this.getPositionToNormal( _vec, _norm );
 
 		const heightDelta = _vec2.subVectors( pos, _vec );
 

test/three/Ellipsoid.test.js

@@ -178,6 +178,30 @@ describe( 'Ellipsoid', () => {
 
 	} );
 
+	it( 'should match Cesium Cartesian to Cartographic results.', () => {
+
+		const HEIGHT_EPSILON = 1e-5;
+		const LAT_LON_EPSILON = 1e-7;
+		const LAT = 40;
+		const LON = - 75;
+		const HEIGHT = 5000;
+		const cart = new Cesium.Cartographic( LON, LAT, HEIGHT );
+
+		c_wgsEllipse.cartographicToCartesian( cart, c );
+		v.set( c.x, c.y, c.z );
+
+		const cesiumResult = new Cesium.Cartographic();
+		c_wgsEllipse.cartesianToCartographic( c, cesiumResult );
+
+		const result = {};
+		wgsEllipse.getPositionToCartographic( v, result );
+
+		compareCoord( result.lon, cesiumResult.longitude, LAT_LON_EPSILON );
+		compareCoord( result.lat, cesiumResult.latitude, LAT_LON_EPSILON );
+		compareCoord( result.height, cesiumResult.height, HEIGHT_EPSILON );
+
+	} );
+
 	it( 'should match the expected elevation.', () => {
 
 		//ellipsoid rotation