[GitHub] [incubator-sedona] kanchanchy commented on a diff in pull request #614: [SEDONA-117] RS_AppendNormalizedDifference Implemented and Raster Notebook Updated

[GitBox <gi...@apache.org>]

kanchanchy commented on code in PR #614:
URL: https://github.com/apache/incubator-sedona/pull/614#discussion_r855804030


##########
sql/src/main/scala/org/apache/spark/sql/sedona_sql/expressions/raster/Functions.scala:
##########
@@ -771,3 +771,45 @@ case class RS_Normalize(inputExpressions: Seq[Expression])
 }
 
 
+/// Append the Normalized Difference between two bands to the image array data as a new band

Review Comment:
   @jiayuasu In current GeoTiff Dataframe of Apache Sedona, each individual band is not a single column. There are two columns related bands: i) nBands abd ii) data. nBands denote the number of bands in a Geotiff image, and data contains all bands in a single array. If there are 4 bands each of length 10, the size of data array is 40, where first 10 elements correspond to first band, second 10 elements correspond to 2nd band....
   I maintained exactly the same structure of current raster dataframe in apache sedona.



-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org