Skip to content

raster_calculator stats to include percentage of valid pixels#434

Merged
phargogh merged 5 commits intonatcap:mainfrom
davemfish:feature/431-raster-stats-valid-percent
Jun 20, 2025
Merged

raster_calculator stats to include percentage of valid pixels#434
phargogh merged 5 commits intonatcap:mainfrom
davemfish:feature/431-raster-stats-valid-percent

Conversation

@davemfish
Copy link
Copy Markdown
Contributor

Fixes #431

This follows GDAL conventions & behavior for what gets written to the band metadata when band.ComputeStatistics(0) is called.

@davemfish davemfish marked this pull request as draft June 18, 2025 21:13
@davemfish
Copy link
Copy Markdown
Contributor Author

The Windows 3.13 wheel build is probably failing for this reason: python/cpython#135151

@davemfish davemfish marked this pull request as ready for review June 19, 2025 18:49
@davemfish davemfish requested a review from phargogh June 19, 2025 18:49
Copy link
Copy Markdown
Member

@phargogh phargogh left a comment

Choose a reason for hiding this comment

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

Thanks @davemfish ! Everything looks good to me.

Reading through the thread in python/cpython#135151, the build failure should be addressed in 3.13.5, which is already out by now but clearly hasn't made its way into github's setup-python action. The build issue here should be resolved once that happens.

@phargogh phargogh merged commit 7c41235 into natcap:main Jun 20, 2025
62 of 63 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.

Feature: raster_calculator to calculate STATISTICS_VALID_PERCENT

2 participants