release v2.0.0 #685

betolink · 2025-05-16T20:19:22Z

v2.0.0 release for Icepyx with Harmony integration replacing the deprecated EGI subsetter. Details in the CHANGELOG, documentation and related issues in Github.

Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Jessica Scheick <[email protected]>

Co-authored-by: GitHub Action <[email protected]>

…tion and unit tests (#609)

…#616)

Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Jessica Scheick <[email protected]>

… group (#629) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

#624)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Co-authored-by: Jessica Scheick <[email protected]>

Allows CI to pass and docs to build without getting stuck on import errors.

… group (#650)

Co-authored-by: GitHub Action <[email protected]>

#654)

… group (#659) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

…in a separate action (#661) Co-authored-by: Matt Fisher <[email protected]>

…ions (#617) Copy integration test workflow from earthaccess Co-authored-by: Jessica Scheick <[email protected]>

Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Co-authored-by: Matt Fisher <[email protected]> Co-authored-by: Trey Stafford <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Theresa Andersen <[email protected]> Co-authored-by: asteiker <[email protected]> Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: Rachel Wegener <[email protected]> Co-authored-by: Wei Ji <[email protected]> Co-authored-by: GitHub Action <[email protected]>

Co-authored-by: Wei Ji <[email protected]>

review-notebook-app · 2025-05-16T20:19:27Z

Check out this pull request on

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

github-actions · 2025-05-16T20:19:33Z

👈 Launch a binder notebook on this branch for commit 14a1372

I will automatically update this comment whenever this PR is modified

weiji14

v2! 🚀🚀

weiji14 · 2025-05-17T01:32:30Z

Don't have permission to merge into protected branch main 😅 Need to wait for someone with higher permissions.

JessicaS11 · 2025-05-21T22:34:44Z

doc/source/user_guide/changelog/v2.0.0.rst

+    API endpoint from the on-premises EGI system to the cloud-based Harmony services.
+    icepyx has been updated to access these new services; Certain capabilities are temporarily going away
+
+    * Data reformatting: users will only get HDF5 files back from the new API.


what if the product is gridded (so natively a netcdf)? I assume this is referring to the fact that reformatting is not available?

Yes, I think if the product is gridded and supported by Harmony the trajectory subsetter cannot do the magic it does with other datasets like "I'd like the result in geoparquet" etc. So whatever the source file is that's the format of the output. Is this correct @asteiker?

so maybe the comment should say "No file reformatting is available for now."

@betolink Yes that is correct, no reformatting is available in Harmony as of now, for any IS2 product. And, none of the gridded (L3B) products are hooked up to Harmony, so no services are available for those products, even subsetting.

How about something like this?

Suggested change

* Data reformatting: users will only get HDF5 files back from the new API.

* Data reformatting: Currently reformatting is not supported for any ICESat-2 products, so users will only get full HDF5 files back from the new Harmony API.

I think we should stay with something along the lines of "no format transcoding is available" because "full HDF5" sounds like not even subsetting is an option. What do you think?

Suggested change

* Data reformatting: users will only get HDF5 files back from the new API.

* Data reformatting: Currently no format transcoding is available for any ICESat-2 products, so users will only get HDF5 files back from the new Harmony API.

Ah, we can't push these changes because the development branch is protected (and I don't think we have permissions to override it). I guess we'll just leave this as is, and update the changelog in a v2.0.1 release.

I've updated this sentence on the GitHub release notes at least - https://github.com/icesat2py/icepyx/releases/tag/v2.0.0. Cheers everyone for making this happen!

doc/source/user_guide/changelog/v2.0.0.rst

weiji14 · 2025-05-23T03:06:28Z

Note that upload of sdist/wheel to TestPyPI succeeded, but upload to PyPI failed due to an attestation file issue. I've manually downloaded the sdist/wheel from TestPyPI (https://test.pypi.org/project/icepyx/2.0.0/#files) and uploaded it to PyPI at https://pypi.org/project/icepyx/2.0.0/#files (hashsums should match).

Also opened a PR at #687 to fix the publish-to-pypi.yml GitHub Action workflow so this won't happen again.

mfisher87 and others added 30 commits September 17, 2024 17:48

Typecheck some API interfaces (#593)

1606dd0

Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Jessica Scheick <[email protected]>

Partially typecheck APIformatting module (#598)

0f2cfeb

Co-authored-by: GitHub Action <[email protected]>

Mark tests which depend on ECS/EGI as expected fail; separate integra…

74ca632

…tion and unit tests (#609)

Bugfix: Move loop_root sentinel value above exhaustive conditionals (…

4e54d0a

…#616)

Remove redundant conditional in APIformatting (#590)

b7e07bf

Type annotate the temporal module (#604)

628ad8f

Co-authored-by: Jessica Scheick <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Jessica Scheick <[email protected]>

add trigger to unit tests upon completion of UML action (#619)

23e3f7e

Bump codecov/codecov-action from 4.5.0 to 4.6.0 in the github-actions…

c454aab

… group (#629) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

add dependabot ignore for non-semantic v1 of repository-traffic-action (

5d668c2

#624)

[pre-commit.ci] pre-commit autoupdate (#631)

50af59d

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

properly apply all dimensions to deeply nested variables (#623)

4539e82

[pre-commit.ci] pre-commit autoupdate (#635)

2511dc6

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

[pre-commit.ci] pre-commit autoupdate (#638)

7605002

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

include quest tile in index.rst (#497)

4eb7aaf

Co-authored-by: Jessica Scheick <[email protected]>

pin to older version of dask (2025.1.0 breaks datashader) (#641)

2f1875a

Allows CI to pass and docs to build without getting stuck on import errors.

make np.datetime64 units "ns" explicitly (#653)

3587c9c

Bump codecov/codecov-action from 4.6.0 to 5.3.1 in the github-actions…

07ff3c0

… group (#650)

Remove existing deprecations from icepyx v1.x (#640)

60d21fc

Co-authored-by: GitHub Action <[email protected]>

[pre-commit.ci] pre-commit autoupdate (#651)

54e3c1d

GitHub Actions: Run unit tests on Python 3.13 (#622)

9591d12

Switch from deprecated chartboost/ruff-action to astral-sh/ruff-action (

c66fd72

#654)

Bump codecov/codecov-action from 5.3.1 to 5.4.0 in the github-actions…

6de304d

… group (#659) Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

[pre-commit.ci] pre-commit autoupdate (#660)

44e7f4e

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Drop support for Python 3.9 and 3.10 (#655)

aeba4eb

Add pytest mark to integration tests requiring download and run them …

82b7239

…in a separate action (#661) Co-authored-by: Matt Fisher <[email protected]>

allow integration tests to run on forks from users with write permiss…

3e30661

…ions (#617) Copy integration test workflow from earthaccess Co-authored-by: Jessica Scheick <[email protected]>

remove pull_request_target trigger for integration tests (#665)

3d43fc6

get variables list from json instead of CMR (#669)

386d73f

traffic and stats update Sept 2024-Mar 2025 (#674)

35211d1

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

betolink and others added 2 commits May 9, 2025 11:35

Add changelog for v2.0.0 (#684)

14a1372

Co-authored-by: Wei Ji <[email protected]>

betolink marked this pull request as ready for review May 16, 2025 21:03

weiji14 changed the title ~~v2.0.0 release~~ release v2.0.0 May 17, 2025

weiji14 approved these changes May 17, 2025

View reviewed changes

weiji14 mentioned this pull request May 21, 2025

Resolve GeoPandas library warnings #681

Merged

JessicaS11 reviewed May 21, 2025

View reviewed changes

doc/source/user_guide/changelog/v2.0.0.rst Show resolved Hide resolved

weiji14 merged commit da1abd4 into main May 23, 2025
60 of 62 checks passed

JessicaS11 mentioned this pull request Oct 10, 2025

get doctests running as part of test actions #713

Draft

	* Data reformatting: users will only get HDF5 files back from the new API.
	* Data reformatting: Currently reformatting is not supported for any ICESat-2 products, so users will only get full HDF5 files back from the new Harmony API.

	* Data reformatting: users will only get HDF5 files back from the new API.
	* Data reformatting: Currently no format transcoding is available for any ICESat-2 products, so users will only get HDF5 files back from the new Harmony API.

release v2.0.0 #685

release v2.0.0 #685

Uh oh!

Conversation

betolink commented May 16, 2025

Uh oh!

review-notebook-app bot commented May 16, 2025

Uh oh!

github-actions bot commented May 16, 2025

Uh oh!

weiji14 left a comment

Choose a reason for hiding this comment

Uh oh!

weiji14 commented May 17, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

betolink May 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

weiji14 May 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

weiji14 commented May 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

8 participants

betolink May 21, 2025 •

edited

Loading

weiji14 May 22, 2025 •

edited

Loading