Skip to content

Comments

feat(sedona-raster-functions): add RS_ConvexHull and item-level CRS to RS_Envelope#597

Merged
Kontinuation merged 3 commits intoapache:mainfrom
Kontinuation:pr6-rs-envelope-convexhull
Feb 13, 2026
Merged

feat(sedona-raster-functions): add RS_ConvexHull and item-level CRS to RS_Envelope#597
Kontinuation merged 3 commits intoapache:mainfrom
Kontinuation:pr6-rs-envelope-convexhull

Conversation

@Kontinuation
Copy link
Member

Summary

  • RS_Envelope: Update to produce geometry output with item-level CRS propagated from the raster's CRS, instead of plain Wkb(Planar, None)
  • RS_ConvexHull (new): Returns the convex hull polygon of a raster from its four corner world coordinates. Unlike RS_Envelope (axis-aligned bounding box), RS_ConvexHull preserves the actual raster footprint for skewed/rotated rasters. Also outputs with item-level CRS.

@Kontinuation Kontinuation requested a review from Copilot February 11, 2026 16:19
Copy link
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

This PR enhances raster geometry functions by propagating CRS metadata and adds a new convex hull function. The changes ensure that geometry outputs preserve the coordinate reference system from input rasters rather than returning plain WKB geometries.

Changes:

  • Modified RS_Envelope to propagate item-level CRS from rasters to output geometries
  • Added RS_ConvexHull function that returns the actual raster footprint (preserving skew/rotation) instead of axis-aligned bounding box
  • Both functions now return geometries with attached CRS metadata using the item_crs pattern

Reviewed changes

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

File Description
rust/sedona-raster-functions/src/rs_envelope.rs Updated to propagate CRS metadata from raster to output WKB geometry
rust/sedona-raster-functions/src/rs_convexhull.rs New function that computes convex hull polygon from raster corner coordinates with CRS propagation
rust/sedona-raster-functions/src/register.rs Registered the new rs_convexhull function
rust/sedona-raster-functions/src/lib.rs Added module declaration for rs_convexhull

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

@Kontinuation Kontinuation force-pushed the pr6-rs-envelope-convexhull branch from 9e2793d to 96e77f3 Compare February 11, 2026 18:25
…o RS_Envelope

Add RS_ConvexHull function that returns the convex hull polygon of a
raster from its four corner coordinates. Update RS_Envelope to produce
geometry output with item-level CRS propagated from the raster's CRS.
@Kontinuation Kontinuation force-pushed the pr6-rs-envelope-convexhull branch from 96e77f3 to aeb70f0 Compare February 13, 2026 03:05
@Kontinuation Kontinuation marked this pull request as ready for review February 13, 2026 04:09
Copy link
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 c519e7c into apache:main Feb 13, 2026
17 checks passed
@paleolimbot paleolimbot added this to the 0.3.0 milestone Feb 19, 2026
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.

2 participants