jiayuasu opened a new pull request, #2633: URL: https://github.com/apache/sedona/pull/2633
## Did you read the Contributor Guide? - Yes, I have read the [Contributor Rules](https://sedona.apache.org/latest/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/latest/community/develop/) ## Is this PR related to a ticket? - Yes, and the PR name follows the format `[GH-XXX] my subject`. Closes #2407 ## What changes were proposed in this PR? `SedonaUtils.display_image()` hangs when passed a raw raster DataFrame (e.g., 1400x800) because `__convert_to_gdf_or_pdf__` attempts to Arrow-serialize the full GridCoverage2D object. This PR adds raster UDT detection to `display_image()`: - Detects raster columns via `typeName() == "rastertype"` - Auto-applies `RS_AsImage()` to each raster column before rendering - Preserves all non-raster columns in the DataFrame - Falls through to the existing `__convert_to_gdf_or_pdf__` + `to_html()` path after conversion Users can now pass a raw raster DataUsers can now pass a raw raster DataUsers can now pass a raw raster DataUsers this patch tested? - Tested manually in Jupyter notebook inside the Sedona Docker container - Verified with raw raster DataFrame (single and multi-column) - Verified with pre-applied `RS_AsImage()` DataFrame (regression check) - Existing test `test_sedonautils.py::test_display_image` continues to pass ## Did this PR include necessary documentation updates? - Yes, I have updated the documentation. Updated `docs/api/sql/Raster-visualizer.md` to show both the new direct raster display (recommended) and the explicit `RS_AsImage` workflow. -- 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]
