-
Notifications
You must be signed in to change notification settings - Fork 310
Thr3mi L1C reader #2908
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
base: main
Are you sure you want to change the base?
Thr3mi L1C reader #2908
Conversation
pepephillips
commented
Sep 24, 2024
- Closes #xxxx
- Tests added
- Fully documented
…to thr3mi_dev Merge branch 'main' of https://github.com/pytroll/satpy into map_dev To take on board latest changes from satpy main before 3MI reader PR
i just looked at the commits and there is an old MAP L1C reader commit in there. The MAP L1C readers were since removed and obviously not part of the final commit, only the new 3MI readers are there. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2908 +/- ##
==========================================
- Coverage 96.23% 96.23% -0.01%
==========================================
Files 398 400 +2
Lines 57325 57520 +195
==========================================
+ Hits 55168 55355 +187
- Misses 2157 2165 +8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Merge branch 'main' of https://github.com/pytroll/satpy into thr3mi_dev
Merge branch 'thr3mi_dev' of https://github.com/pepephillips/satpy into thr3mi_dev
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
Pull Request Test Coverage Report for Build 14907297120Details
💛 - Coveralls |
…erage for exception handling
@mraspaud, this is a very basic 3mi L1C reader for the native 3mi normalised reflectances in the 1D data format. As a result, scene.show() will not work as the data is still 1D (but I can hopefully update that later). However, once resampled, you can plot the projected data ok using show(). I suspect my coding is too basic and I hope someone can improve on it downstream.... I hope to include conversion to reflectance/radiance and colour composites (including polarised composites) too. |