Skip to content

Comments

Modify tolerance to be an argument in rio.warp.reproject#3325

Merged
sgillies merged 1 commit intorasterio:mainfrom
rhugonnet:add_tolerance_arg_reproject
Apr 5, 2025
Merged

Modify tolerance to be an argument in rio.warp.reproject#3325
sgillies merged 1 commit intorasterio:mainfrom
rhugonnet:add_tolerance_arg_reproject

Conversation

@rhugonnet
Copy link
Contributor

This PR makes tolerance a user-defined argument in rio.warp.reproject.

I matched the argument description of WarpedVRTReaderBase for consistency:

tolerance : float, optional

Resolves #3324

Copy link
Member

@sgillies sgillies left a comment

Choose a reason for hiding this comment

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

Thanks @rhugonnet !

@rhugonnet
Copy link
Contributor Author

Looks like failing CI tests are unrelated.
@sgillies Thanks for all the work!

@sgillies sgillies merged commit aa08101 into rasterio:main Apr 5, 2025
12 of 22 checks passed
@snowman2 snowman2 modified the milestones: 1.4.4, 1.5.0 Nov 6, 2025
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.

Tolerance argument of rio.warp.reproject cannot be modified by user

3 participants