Skip to content
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

ENH: Bring in extended list of supported message structures from MetPy for Level2 reader #163

Open
jthielen opened this issue Mar 15, 2024 · 1 comment

Comments

@jthielen
Copy link

As referenced in #158 and discussed with @mgrover1 in the MetPy telecon yesterday, there is still some relevant code in main...jthielen:xradar:nexrad-level2 that may be helpful, namely that of a reconciled list of message structures combining those from both Py-ART's and MetPy's. Given that prior effort used MetPy's bespoke NamedStruct and BitField objects to handle this, there would likely be a quite a bit of refactoring needed (going either direction..modifying the reconciled lists to match the current xradar approach, or updating the xradar approach to use these MetPy-style objects). This issue can hopefully track this particular facet of the overall effort of #40.

@kmuehlbauer
Copy link
Collaborator

Thanks @jthielen for pushing this here.

I can have a look what would be necessary to incorporate things like BitField into the current structure.

I'm also interested to check if and how NamedStruct can be utilized for the current processing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants