jesspav commented on code in PR #405:
URL: https://github.com/apache/sedona-db/pull/405#discussion_r2587422363
##########
rust/sedona-raster-functions/src/rs_worldcoordinate.rs:
##########
@@ -77,6 +90,19 @@ fn rs_rastertoworldcoordx_doc() -> Documentation {
.build()
}
+fn rs_rastertoworldcoord_doc() -> Documentation {
+ Documentation::builder(
+ DOC_SECTION_OTHER,
+ "Returns the grid coordinate of the given world coordinates as a
Point.".to_string(),
+ "RS_RasterToWorldCoord(raster: Raster, x: Integer, y:
Integer)".to_string(),
+ )
+ .with_argument("raster", "Raster: Input raster")
+ .with_argument("x", "Integer: Column x into the raster")
+ .with_argument("y", "Integer: Row y into the raster")
Review Comment:
I generally find copilot amazing for finding doc improvements, but this one
does not seem correct. Sticking with the row/column terminology that is
elsewhere.
--
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]
