-
Notifications
You must be signed in to change notification settings - Fork 17
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
Migrating mpl reader from ACT to xradar #159
Comments
@zssherman Great initiative! It looks like MPL is also some binary format with neat header structures. The sigmet/iris reader heavily uses these kind of structured decoding. This is also what #158 is trying to achieve for nexrad level2. Maybe we can discuss on next open radar meeting which steps are necessary to get a prototype reader ready. |
@kmuehlbauer That sounds good to me! |
@zssherman Since there wasn't much time yesterday I'll follow up with some ideas/pointers here.
I'm not really sure how to handle the sidecar files, but we might just search/recognize them and directly read/decode as binary blobs (when without header). For the main file the idea would be to use xradar/xradar/io/backends/nexrad_level2.py Lines 137 to 151 in 56a9ca1
Then the header could be directly extracted using the machinery from the iris/sigmet reader: xradar/xradar/io/backends/nexrad_level2.py Lines 187 to 190 in 56a9ca1
For this the header structure needs some special layout, where decoding information can be attached into the xradar/xradar/io/backends/nexrad_level2.py Lines 725 to 733 in 56a9ca1
The actual data might be read with dedicated functions (eg names like xradar/xradar/io/backends/nexrad_level2.py Lines 576 to 608 in 56a9ca1
This xradar/xradar/io/backends/nexrad_level2.py Line 1227 in 56a9ca1
This is then used in the XarrayStore to provide Variables/Coordinates xradar/xradar/io/backends/nexrad_level2.py Line 1310 in 56a9ca1
xradar/xradar/io/backends/nexrad_level2.py Line 1328 in 56a9ca1
I hope this does at least make some sense and you could give it a try. |
After the ACT dev call, we discussed on how moving the MPL reader to xradar would be more fitting:
ARM-DOE/ACT#806
I can give this a shot. I will just need to learn the backends of Xarray first.
The text was updated successfully, but these errors were encountered: