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
Check whether time dimension exists for timeseries #1173
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1173 +/- ##
==========================================
+ Coverage 89.60% 89.95% +0.34%
==========================================
Files 200 208 +8
Lines 29484 30596 +1112
==========================================
+ Hits 26420 27523 +1103
- Misses 3064 3073 +9
Continue to review full report at Codecov.
|
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.
LGTM. Do we haves tests for multiscene? In which case a small test for this would be nice to have
@mraspaud I've added the test to the part of |
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.
LGTM
@djhoese Is this PR fine enough? |
@zxdawn Sorry I forgot to merge this. |
If
time
dimension exists, we can blend them directly.flake8 satpy
AUTHORS.md
if not there already