Skip to content

Conversation

@prantogg
Copy link
Contributor

Did you read the Contributor Guide?

Is this PR related to a ticket?

  • Yes, and the PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?

  • Uses rasterizeGeomExtent logic from Rasterization class before clipping raster
  • This prevents invalid dimensions (height<1; width<1) error

How was this patch tested?

  • Passes new and existing tests

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

@prantogg prantogg requested a review from jiayuasu as a code owner May 21, 2025 16:15
@prantogg prantogg requested review from Kontinuation and jiayuasu and removed request for jiayuasu May 21, 2025 16:15
@jiayuasu jiayuasu added the bug label May 21, 2025
@jiayuasu jiayuasu added this to the sedona-1.8.0 milestone May 21, 2025
@jiayuasu jiayuasu merged commit 354b55d into apache:master May 21, 2025
37 checks passed
jiayuasu pushed a commit that referenced this pull request May 30, 2025
…xel size (#1952)

* profiling1

* profiling4

* profiling...

* profiling 8

* remove redundant cropping

* fix lenient mode

* Use rasterizeGeomExtent from Rasterization class

* add comment

* add test for small AOI geometries

* make rasterizeGeomExtent a protected method

* add test geotif

* undo pom changes

* Address comments

* spotless fix
Kontinuation pushed a commit to Kontinuation/sedona that referenced this pull request Jan 21, 2026
…xel size (apache#1952)

* profiling1

* profiling4

* profiling...

* profiling 8

* remove redundant cropping

* fix lenient mode

* Use rasterizeGeomExtent from Rasterization class

* add comment

* add test for small AOI geometries

* make rasterizeGeomExtent a protected method

* add test geotif

* undo pom changes

* Address comments

* spotless fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants