Skip to content

support bigtiff into geotiff reader#656

Merged
sandorkertesz merged 4 commits intodevelopfrom
feat/add_bigtiff_support
Apr 4, 2025
Merged

support bigtiff into geotiff reader#656
sandorkertesz merged 4 commits intodevelopfrom
feat/add_bigtiff_support

Conversation

@Oisin-M
Copy link
Contributor

@Oisin-M Oisin-M commented Mar 19, 2025

Close #655

@FussyDuck
Copy link

FussyDuck commented Mar 19, 2025

CLA assistant check
All committers have signed the CLA.

@codecov-commenter
Copy link

codecov-commenter commented Mar 19, 2025

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 90.15%. Comparing base (cfe11c2) to head (db5d27e).

Files with missing lines Patch % Lines
tests/readers/test_geotiff_reader.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #656   +/-   ##
========================================
  Coverage    90.14%   90.15%           
========================================
  Files          159      159           
  Lines        12172    12173    +1     
  Branches       595      595           
========================================
+ Hits         10973    10974    +1     
  Misses        1017     1017           
  Partials       182      182           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sandorkertesz
Copy link
Collaborator

@Oisin-M, thank you very much for this fix. Can I ask you to add a test for it e.g. in "tests/readers/test_geotiff_reader.py" with an example file as small as possible?

@sandorkertesz
Copy link
Collaborator

@Oisin-M, thank you for adding the test. Once updated with develop it can be merged.

@Oisin-M
Copy link
Contributor Author

Oisin-M commented Apr 3, 2025

Hi @sandorkertesz, that's now up to date with develop

@sandorkertesz sandorkertesz merged commit d79521b into develop Apr 4, 2025
112 of 113 checks passed
@Oisin-M Oisin-M deleted the feat/add_bigtiff_support branch November 12, 2025 12:53
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.

Cannot read BigTIFF with GeoTIFF reader

4 participants