Skip to content

feat(sedona-gdal): add raster operations and vrt support#698

Merged
Kontinuation merged 16 commits intoapache:mainfrom
Kontinuation:feat/sedona-gdal-safe-ops
Apr 1, 2026
Merged

feat(sedona-gdal): add raster operations and vrt support#698
Kontinuation merged 16 commits intoapache:mainfrom
Kontinuation:feat/sedona-gdal-safe-ops

Conversation

@Kontinuation
Copy link
Copy Markdown
Member

@Kontinuation Kontinuation commented Mar 9, 2026

Summary

  • add VRT dataset support plus rasterize, rasterize-affine, and polygonize wrappers
  • layer higher-level raster operations on top of the dataset/raster/vector wrapper stack
  • remove raster wrapper re-export aliases so call sites import concrete module paths directly

Stack

Follow-up since primitives merged

  • rebase onto the updated safe-core branch
  • use GdalApi::last_null_pointer_err instead of constructing GdalError::NullPointer directly
  • keep explicit module-path imports after removing raster re-export aliases

Testing

  • cargo test -p sedona-gdal
  • cargo clippy -p sedona-gdal -- -D warnings

@Kontinuation Kontinuation force-pushed the feat/sedona-gdal-safe-ops branch 7 times, most recently from b7fa46a to e1d4617 Compare March 13, 2026 06:38
@Kontinuation Kontinuation force-pushed the feat/sedona-gdal-safe-ops branch 2 times, most recently from 1b81e94 to 230774c Compare March 19, 2026 14:44
@Kontinuation Kontinuation force-pushed the feat/sedona-gdal-safe-ops branch from 230774c to e92db8b Compare March 26, 2026 14:38
@Kontinuation Kontinuation marked this pull request as ready for review March 27, 2026 01:27
@Kontinuation Kontinuation requested a review from Copilot March 27, 2026 01:27
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds new GDAL raster operation wrappers and VRT dataset support to the sedona-gdal safe wrapper layer, building on the existing dataset/raster/vector abstractions.

Changes:

  • Introduces VrtDataset/VrtRasterBand wrappers with VRT creation, band add, simple source, and nodata helpers.
  • Adds raster operations: rasterize, rasterize_affine (fast geotransform-only path), and polygonize, with tests.
  • Exposes the new modules via raster.rs and lib.rs.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
c/sedona-gdal/src/vrt.rs New VRT dataset and band wrappers + tests
c/sedona-gdal/src/raster/rasterize.rs New GDALRasterizeGeometries wrapper + options + tests
c/sedona-gdal/src/raster/rasterize_affine.rs New fast affine rasterize wrapper using custom transformer + tests
c/sedona-gdal/src/raster/polygonize.rs New GDALPolygonize wrapper + options + tests
c/sedona-gdal/src/raster.rs Exposes new raster modules
c/sedona-gdal/src/lib.rs Exposes new vrt module

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread c/sedona-gdal/src/vrt.rs Outdated
Comment thread c/sedona-gdal/src/vrt.rs
Comment thread c/sedona-gdal/src/vrt.rs
Comment thread c/sedona-gdal/src/raster/rasterize_affine.rs
Comment thread c/sedona-gdal/src/raster/rasterize_affine.rs
Comment thread c/sedona-gdal/src/raster/rasterize_affine.rs
Copy link
Copy Markdown
Member

@paleolimbot paleolimbot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Kontinuation Kontinuation merged commit f110d44 into apache:main Apr 1, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants