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

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

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


##########
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:
   I am kind of confused by this new function. In the current Sedona Raster DataFrame, each individual band is a single column. So if these GeoTiffs have 9 bands, our raster dataframe will have 9 columns.
   
   But in your case, it seems that you put all bands in one column? This is different from what the GeoTiff reader does.



-- 
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