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
Bugfix msg native reader #265
Conversation
Signed-off-by: Adam.Dybbroe <adam.dybbroe@smhi.se>
Signed-off-by: Adam.Dybbroe <adam.dybbroe@smhi.se>
@@ -33,6 +33,8 @@ | |||
import numpy as np | |||
|
|||
import xarray as xr | |||
import dask.array as da | |||
from satpy import CHUNK_SIZE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
F401 'satpy.CHUNK_SIZE' imported but unused
We are not quite ready with this PR yet. Sorry. Will fix the tests first! |
Signed-off-by: Adam.Dybbroe <adam.dybbroe@smhi.se>
Signed-off-by: Adam.Dybbroe <adam.dybbroe@smhi.se>
Codecov Report
@@ Coverage Diff @@
## develop #265 +/- ##
===========================================
- Coverage 63.8% 63.76% -0.05%
===========================================
Files 112 112
Lines 13861 13877 +16
===========================================
+ Hits 8844 8848 +4
- Misses 5017 5029 +12
Continue to review full report at Codecov.
|
…/satpy into bugfix-msg-native-reader
satpy/readers/native_msg.py
Outdated
|
||
# The available channels are only known after the header | ||
# has been read, after that we know what the indices are for each channel | ||
self.available_channels = dict(zip(CHANNEL_LIST,[False]*len(CHANNEL_LIST))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E231 missing whitespace after ','
Signed-off-by: Adam.Dybbroe <adam.dybbroe@smhi.se>
…ative format Signed-off-by: Adam.Dybbroe <adam.dybbroe@smhi.se>
I believe we are ready now, and you can merge when ever it is suitable |
We merged develop into this and pushed again. Should be ready to merge. |
Fixing the MSG SEVIRI native format reader, and make it Dask compatible.
The changes previously done by Colin was not fully completed. Now, the reader should handle subsetted datasets.
git diff origin/develop **/*py | flake8 --diff