prantogg commented on code in PR #1122: URL: https://github.com/apache/sedona/pull/1122#discussion_r1394741894
########## docs/api/sql/Raster-operators.md: ########## @@ -1548,21 +1548,28 @@ Output: ### RS_Value -Introduction: Returns the value at the given point in the raster. -If no band number is specified it defaults to 1. +Introduction: Returns the value at the given point in the raster. Format: -`RS_Value (raster: Raster, point: Geometry)` - Review Comment: Have kept `RS_Value (raster: Raster, point: Geometry)` signature and reverted changes to R tests. Note: - Only Point geometry variant of RS_Value and RS_Values can take band as optional parameter - Same currently not possible for the integer variant because of conflicting function signatures as shown below, `RS_Value (raster: Raster, colX: Integer, colY: Integer)` `RS_Value (raster: Raster, point: Geometry, band: Integer)` -- 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]
