jiayuasu commented on PR #612: URL: https://github.com/apache/incubator-sedona/pull/612#issuecomment-1101898005
@kanchanchy This PR looks awesome. I have a small request regarding the CRS in GeoTIFF reader and writer. Currently, it is fixed to (1) in the reader, read from the original data files and read to EPSG 4326 (2) in the writer, write to EPSG 4326. This creates some troubles to the end user. Can you provide some optional parameters to the user such as This will write data to the desired the CRS ``` sparkSession.read.format("geotiff").option("writeFromCRS", "epsg:XXX").save("YOUR_PATH") sparkSession.read.format("geotiff").option("writeToCRS", "epsg:XXX").save("YOUR_PATH") ``` If the original GeoTiff data files does not have CRS inside, then use the following ``` sparkSession.read.format("geotiff").option("readFromCRS", "epsg:XXX").load("YOUR_PATH") sparkSession.read.format("geotiff").option("readToCRS", "epsg:XXX").load("YOUR_PATH") ``` If the original GeoTiff data files have CRS inside, `readFromCRS` will not be in effect. Both of reader and writer should have an optional parameter which is option("disableErrorInCRS", "true"). This will be the parameter used here: https://github.com/apache/incubator-sedona/blob/master/sql/src/main/scala/org/apache/spark/sql/sedona_sql/expressions/Functions.scala#L288 The default value should be false -- 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-unsubscr...@sedona.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org