-
Notifications
You must be signed in to change notification settings - Fork 288
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
Fix the avhrr_l1b_gaclac to support angles, units and avhrr variants #759
Conversation
Codecov Report
@@ Coverage Diff @@
## master #759 +/- ##
==========================================
- Coverage 81.61% 81.56% -0.06%
==========================================
Files 157 157
Lines 22636 22652 +16
==========================================
Hits 18475 18475
- Misses 4161 4177 +16
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #759 +/- ##
==========================================
+ Coverage 81.61% 81.85% +0.23%
==========================================
Files 157 158 +1
Lines 22636 22734 +98
==========================================
+ Hits 18475 18608 +133
+ Misses 4161 4126 -35
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
return xr.DataArray(da.from_array(data, chunks=(chunk_lines, chunk_cols)), | ||
dims=['y', 'x'], attrs=info) | ||
res = xr.DataArray(da.from_array(data, chunks=(chunk_lines, chunk_cols)), | ||
dims=['y', 'x'], attrs=info) |
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.
Adding the scanline timestamps as additional y-coordinate would be nice here. But since this PR seems urgent and the CF writer does not completely support timestamps (yet), we can do that later. Added it to my TODO 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.
I can try to add it tonight. Is the cf writer crashing if we add the times atm ?
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.
Done
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.
Just a couple things I noticed that weren't necessarily something you did. A few typos otherwise.
@@ -25,6 +26,7 @@ datasets: | |||
calibration: | |||
reflectance: | |||
standard_name: reflectance |
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.
I believe standard_names should be toa_bidirectional_reflectance
and toa_brightness_temperature
. @adybbroe could maybe verify that I'm not making things up. I don't remember what the radiance is supposed to be.
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.
Yes, good point.
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.
Although I checked a few other readers, we have reflectance
and brightness_temperature
there. So it might need another PR to fix all of these
satpy/readers/avhrr_l1b_gaclac.py
Outdated
# Author(s): | ||
|
||
# Martin Raspaud <martin.raspaud@smhi.se> | ||
# Copyright (c) 2009-2019 Pytroll developpers |
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.
Typo developers
@@ -0,0 +1,136 @@ | |||
#!/usr/bin/env python | |||
# -*- coding: utf-8 -*- | |||
# Copyright (c) 2009-2019 Pytroll developpers |
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.
Same typo
0474df5
to
1c53695
Compare
This small PR makes angles available, and sets the units and avhrr channels properly
git diff origin/master -- "*py" | flake8 --diff