Kontinuation commented on code in PR #589:
URL: https://github.com/apache/sedona-db/pull/589#discussion_r2797706521
##########
rust/sedona-schema/src/raster.rs:
##########
@@ -88,19 +88,22 @@ impl RasterSchema {
/// Band data type enumeration for raster bands.
///
-/// Only supports basic numeric types.
-/// In future versions, consider support for complex types used in
-/// radar and other wave-based data.
+/// Ordinals match GDALDataType for real-valued pixel types only.
+/// GDT_Unknown (0) and complex types (CInt16=8, CInt32=9, CFloat32=10,
CFloat64=11)
+/// are intentionally omitted.
Review Comment:
We don't need `Other` variant for now since we decided to not enforce the
values of `BandDataType` to match GDAL. I believe that it is a better design.
--
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]
