-
Notifications
You must be signed in to change notification settings - Fork 15
Sprint: Updating two GOES and three marine converters for naming conventions #1121
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
Merged
srherbener
merged 31 commits into
feature/sprint-ioda-converters
from
feature/sprint-ioda-converters-greg1
Jan 3, 2023
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
bf75378
update GOES converter to data conventions
gthompsnWRF c76dfad
updated some marine converters for dateTime and naming conventions
gthompsnWRF d22f8a1
updated MetaData group vars to new naming convention
gthompsnWRF 5465b36
switch preqc from float to int
gthompsnWRF 872e4e2
quick change of Location to 32-bit integer, not 64-bit
gthompsnWRF 6651f4d
changing wording to albedo from reflectance factor
gthompsnWRF 8225561
fix latlon Location dimension to int32 type, not int64
gthompsnWRF 2470c66
fix latlon Location dimension to int32 type, not int64
gthompsnWRF f1783b7
fix indent for coding norm
gthompsnWRF 69de9ed
another coding norm fix
gthompsnWRF c9e6041
import fix that was added to develop is copied here also
gthompsnWRF f5b4f9b
updating few marine converters to use dateTime properly
gthompsnWRF db36535
updating a couple test reference files
gthompsnWRF 17d7f37
updating smap land converters and test reference files for naming con…
gthompsnWRF 399d736
amsr2 converter and test reference file updated for naming conventions
gthompsnWRF 7196ca8
more marine and land converters updated naming convention
gthompsnWRF fa370ee
updated omi O3 converter for naming conventions
gthompsnWRF ed0c1d4
update NSIDC ice fraction converter to naming conventions
gthompsnWRF 6a86620
quick fixes for coding norms
gthompsnWRF fa0596b
updating mopitt CO converter to naming conventions
gthompsnWRF 6a34890
fix MODIS AOD converter and adopt naming conventions
gthompsnWRF 1db00df
update MLS ozone converter to naming conventions
gthompsnWRF 6839b75
updating AVHRR converter to naming conventions
gthompsnWRF 57af4ac
update 1 of 2 sig-wave-height converters to naming convention
gthompsnWRF e73a61b
update copernicus absoluteDynamicTopography convert to naming convent…
gthompsnWRF 3771c01
fix a coding norm problem
gthompsnWRF f4c8a82
fix overindent coding norm
gthompsnWRF 91613c8
change ncdiff tolerance check on some (to match develop branch)
gthompsnWRF afbca6c
update GNSSRO python converter to naming convention
gthompsnWRF b8947cf
updating testoutput file for change to fillvalue
gthompsnWRF 778b1ba
empty commit
BenjaminRuston File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
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.
Why switch to pyhdf API? What problem is this solving?
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.
Because I had to. The input file is hdf4, not hdf5/netcdf, so I had to switch. This is basically coming from a develop branch change that someone made - so I just followed it thru into the sprint branch so it functions again.
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.
also found this strange was this something from the aerosol group? but if
pyhdfis supported in the stack would potentially get this through and then have the aerosol group review and hopefully consolidate and minimize the number of libraries usedThere 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.
@gthompsnJCSDA thanks for the explanation. pyhdf is supported/included in spack-stack. It would be nice in the future to get providers updated from hdf4 to netcdf/hdf5, but that is certainly outside the scope of this PR.