IO Updates #1476
Merged
IO Updates #1476
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dopplershift
added
Area: IO
|
This pull request introduces 1 alert when merging 7f51825 into 071c700 - view on LGTM.com new alerts:
|
|
Marking draft because I want to take a shot at using |
dopplershift
force-pushed
the
io-update
branch
from
7f51825
to
ad3cf42
Compare
|
This pull request introduces 1 alert when merging ad3cf42 into 931c114 - view on LGTM.com new alerts:
|
dopplershift
force-pushed
the
io-update
branch
from
ad3cf42
to
62f174a
Compare
|
This pull request introduces 1 alert when merging 62f174a into cdeb8c7 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging f919a42 into cdeb8c7 - view on LGTM.com new alerts:
|
This just requires a one-line change in open_as_needed to force conversion to string. This is a nice addition for our pure Python 3 world.
This was found based on information in SCN20-03; channel 60 was previously unknown, and not even found by Unidata's ingest code.
Also fixes __repr__ to work for GSM.
dopplershift
force-pushed
the
io-update
branch
4 times, most recently
from
75cfa40
to
81891ed
Compare
dopplershift
force-pushed
the
io-update
branch
from
81891ed
to
3441003
Compare
|
This pull request introduces 1 alert when merging 3441003 into 69f86f9 - view on LGTM.com new alerts:
|
This, for instance, makes it possible to use Level2File with the realtime chunks available on AWS.
Update product specific fields to include new metadata relating to SAILS/MRLE.
Puts a basic lookup table in the base mapper class; reduces some code duplication as well as silences some PyCharm warnings.
Add some sample test files, add support for a couple new products, and fix a bug in the mapping of width data.
Added in ORPG Build 19.0.
Pulled new classes from the ICD.
Two main changes: * Add read_array function for reading data directly into a numpy array using from_buffer. This is noticably faster (~25%) * Don't assume any blocks are present or their order--this is technically more correct, as well as more robust to future format adjustments.
Update it to use Python 3's int.from_bytes() method.
This allows them to be accessed outside methods.
This paves the way towards writing instances of NamedStructs.
dopplershift
force-pushed
the
io-update
branch
from
3441003
to
efcd1c3
Compare
|
This pull request introduces 1 alert when merging efcd1c3 into 69f86f9 - view on LGTM.com new alerts:
|
dcamron
approved these changes
Oct 5, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description Of Changes
Some clean-ups and updates for GINI and NEXRAD Levels 2 and 3. Definitely a bunch of updates due to NEXRAD ORPG Build 19.
pathlib.Pathobjects (starts to address Check support for pathlib.Path #224)Checklist
from_bytes#1367