Skip to content

DOC: Contributing instructions, BytesIO, switch from rasterio, GDAL config, & external courses#910

Merged
snowman2 merged 19 commits intocorteva:masterfrom
remi-braun:rasterio_doc
Mar 3, 2026
Merged

DOC: Contributing instructions, BytesIO, switch from rasterio, GDAL config, & external courses#910
snowman2 merged 19 commits intocorteva:masterfrom
remi-braun:rasterio_doc

Conversation

@remi-braun
Copy link
Contributor

@remi-braun remi-braun commented Feb 25, 2026

@remi-braun
Copy link
Contributor Author

I tried to fix some open documentation issues and propagate some user questions.
This can of course be amended by anybody, it's more of a draft right now.

Other questions :

  • do you want to switch to another sphinx them?
  • do you want to switch to markdown files instead of rst?

@snowman2 snowman2 added the documentation Documentation related issue label Feb 25, 2026
@remi-braun
Copy link
Contributor Author

remi-braun commented Feb 25, 2026

For the rasterio / rioxarray equivalences, it could be great to explain why some attributes in rasterio are becoming methods when others don't (i.e. transform). It's to preserve the lazy evaluation inherited from xarray and not recomputing on the fly these attributes when changing the raster (i.e. cropping, reprojecting...)?

Co-authored-by: Alan D. Snow <alansnow21@gmail.com>
Co-authored-by: Alan D. Snow <alansnow21@gmail.com>
@snowman2
Copy link
Member

snowman2 commented Mar 3, 2026

@remi-braun, there are some linting issues you can fix by running pre-commit run -a.

@snowman2 snowman2 changed the title Improve documentation DOC: Contributing instructions, BytesIO, switch from rasterio, GDAL config, & external courses Mar 3, 2026
@snowman2 snowman2 merged commit 2e5cf73 into corteva:master Mar 3, 2026
14 of 21 checks passed
@snowman2
Copy link
Member

snowman2 commented Mar 3, 2026

Thanks @remi-braun 👍

@codecov
Copy link

codecov bot commented Mar 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.55%. Comparing base (ae6635a) to head (74e1faf).
⚠️ Report is 199 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #910      +/-   ##
==========================================
- Coverage   95.90%   95.55%   -0.35%     
==========================================
  Files          12       18       +6     
  Lines        1735     2094     +359     
==========================================
+ Hits         1664     2001     +337     
- Misses         71       93      +22     

☔ 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.

@remi-braun
Copy link
Contributor Author

Should I create issues relatively to the questions I had in the beginning of this PR?

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

Labels

documentation Documentation related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants