Copilot commented on code in PR #2690: URL: https://github.com/apache/sedona/pull/2690#discussion_r2887726811
########## docs/api/snowflake/vector-data/Affine-Transformations/ST_Translate.md: ########## @@ -26,6 +26,8 @@ If the geometry is 2D, and a deltaZ parameter is specified, no change is done to If the geometry is empty, no change is done to it. If the given geometry contains sub-geometries (GEOMETRY COLLECTION, MULTI POLYGON/LINE/POINT), all underlying geometries are individually translated. + + Format: `ST_Translate(geometry: geometry, deltaX: deltaX, deltaY: deltaY, deltaZ: deltaZ)` Review Comment: The Snowflake doc's format signature lists parameter names as if they were types (`deltaX: deltaX`, etc.). This appears inconsistent with the other Snowflake affine transformation docs (which use `Double`) and with the SQL/Flink variants. Please correct the signature to use the actual parameter types (e.g., `deltaX: Double`, `deltaY: Double`, `deltaZ: Double`). ```suggestion Format: `ST_Translate(geometry: Geometry, deltaX: Double, deltaY: Double, deltaZ: Double)` ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
