-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix excessive warnings when writing COGs #204
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #204 +/- ##
========================================
Coverage 94.91% 94.91%
========================================
Files 33 33
Lines 5663 5663
========================================
Hits 5375 5375
Misses 288 288 ☔ View full report in Codecov by Sentry. |
@robbibt please add a write to file test to here: Lines 43 to 59 in 8b5c56f
|
@Kirill888 Is that what you had in mind? I just tested it wrote out and could be read back in. |
Based in discussion in #203, this implements a fix to exclude "width", "height", "count", "dtype", "crs", "transform", "nodata" kwargs from
rio_opts
, matching the approach taken for in-memory COGs.