-
Notifications
You must be signed in to change notification settings - Fork 14
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
Release/2.3.0 #140
Merged
Merged
Release/2.3.0 #140
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
* Add initial poetry setup guidance to the README * Update CHANGELOG "unreleased" with issue 85
* make note in README of `-E harmony` install option for tests * Update CHANGELOG.md Co-authored-by: dkaufma3 <[email protected]>
# Conflicts: # CHANGELOG.md # cmr/uat_associations.txt # pyproject.toml
* Add extra line of logic to catch timedelta time cases * Updated Changelog * Add logic to handle time attributes for he5 time converted files * Linted code
* Add extra line of logic to catch timedelta time cases * Updated Changelog * Add logic to handle time attributes for he5 time converted files * Linted code * Added extra logic for compute time vars for cases without any variables * change back chunking * Update Changelog
* Add extra line of logic to catch timedelta time cases * Updated Changelog * Add logic to handle time attributes for he5 time converted files * Linted code * Fix for ncdataset rename deprication - test writing to follow * Remove unnecessary comments, changing to issue 119 * Added test and linted for duplicate dimension name change * Updated Changelog.md
# Conflicts: # CHANGELOG.md # pyproject.toml
* fix way xarray open granules that have as a time unit * fix pylint * change function to use original function if can parse only change units if we can not parse * make xarray override into its own function * add test for override_decode_cf_datetime function * disable pyline one line instead of global * Update podaac/subsetter/subset.py Co-authored-by: Frank Greguska <[email protected]>
* Add variable leading slash flexibility * Add tests back to test file * changelog added and updated * Update podaac/subsetter/subset.py Co-authored-by: Frank Greguska <[email protected]> * update Syntax * resolve conflict Co-authored-by: nlensse1 <[email protected]> Co-authored-by: Frank Greguska <[email protected]>
…datasets (#135) * Update tests to use parameterized pytest. Some tests failing on new TEMPO test dataset * Replace / in all var names if the dataset is grouped * perform flattening as necessary in test suite (#130) * move methods for flattening netcdf and hdf group structures to separate module * feature/PODAAC-5065 (#129) * fix way xarray open granules that have as a time unit * fix pylint * change function to use original function if can parse only change units if we can not parse * make xarray override into its own function * add test for override_decode_cf_datetime function * disable pyline one line instead of global * Update podaac/subsetter/subset.py Co-authored-by: Frank Greguska <[email protected]> * add missing parameter to docstring * typo in docstring * extract netcdf opening procedure from beginning of `subset() into a new function * update tests to use netcdf opening wrapper function, to prevent errors with tempo data * /version 2.3.0-alpha.5 * update `test_specified_variables()` to use netcdf opening wrapper function in multiple places to prevent errors with tempo data * cosmetic * clean up comment and use 'decode_times'=True for test * feature/issue 126 (#131) * Add variable leading slash flexibility * Add tests back to test file * changelog added and updated * Update podaac/subsetter/subset.py Co-authored-by: Frank Greguska <[email protected]> * update Syntax * resolve conflict Co-authored-by: nlensse1 <[email protected]> Co-authored-by: Frank Greguska <[email protected]> * /version 2.3.0-alpha.6 * Update build-pipeline.yml * /version 2.3.0-alpha.7 * Merge changes from origin/develop * Merge changes from issues/127 Co-authored-by: sliu008 <[email protected]> Co-authored-by: Frank Greguska <[email protected]> Co-authored-by: l2ss-py bot <[email protected]> Co-authored-by: Nick Lenssen <[email protected]> Co-authored-by: nlensse1 <[email protected]> * Run verification on issue branches * Updated changelog * add module docstring * fix import statements * fix import statements * Fix tests Co-authored-by: Daniel Kaufman <[email protected]> Co-authored-by: sliu008 <[email protected]> Co-authored-by: l2ss-py bot <[email protected]> Co-authored-by: Nick Lenssen <[email protected]> Co-authored-by: nlensse1 <[email protected]> Co-authored-by: danielfromearth <[email protected]>
* added snyk blocking step to the github build workflow * added snyk blocking step to the github build workflow * added snyk blocking step to the github build workflow * added snyk blocking step to the github build workflow * added snyk blocking step to the github build workflow Co-authored-by: Jonathan M Smolenski <[email protected]>
* typo * rearrange docstring parameters to match function signature * add "TODO" comments for docstring or typing components that aren't quite matching * add type annotations for dimension_cleanup.py * add type annotations for group_handling.py * add type annotations for xarray_enhancements.py * revert 't[T]uple' annotation to python=3.8 using typing import * revert 'l[L]ist' annotation to python=3.8 using typing import * revert 'l[L]ist' annotation to python=3.8 using typing import * add type annotations for subset.py * add type annotations for subset_harmony.py * add type annotations for run_subsetter.py * add further type annotations for subset.py
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
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.
[2.3.0]
Added
for variables to not have leading slash in the front
Changed
Deprecated
Removed
Fixed
seconds since 2000-1-1 0:0:0 0
as a time unit.subset
will now have/
replaced byGROUP_DELIM
so they can be located in flattened datasetsSecurity