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

Release/2.3.0 #140

Merged
merged 47 commits into from
Jan 25, 2023
Merged

Release/2.3.0 #140

merged 47 commits into from
Jan 25, 2023

Conversation

jamesfwood
Copy link
Collaborator

[2.3.0]

Added

  • issue/126: Added flexibility to variable subsetting
    for variables to not have leading slash in the front
  • issue/136: Added type annotations throughout the package code

Changed

Deprecated

Removed

Fixed

  • PODAAC-5065: integration with SMAP_RSS_L2_SSS_V5, fix way xarray open granules that have seconds since 2000-1-1 0:0:0 0 as a time unit.
  • issue/127: Fixed bug when subsetting variables in grouped datasets. Variable names passed to subset will now have / replaced by GROUP_DELIM so they can be located in flattened datasets

Security

l2ss-py bot and others added 30 commits August 29, 2022 16:19
* 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
l2ss-py bot and others added 17 commits November 9, 2022 00:28
* 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
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@jamesfwood jamesfwood merged commit 64ab511 into main Jan 25, 2023
@jamesfwood jamesfwood deleted the release/2.3.0 branch January 25, 2023 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants