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

Ab/upgrade 2024.01.23 fixes #119

Closed
wants to merge 4 commits into from

Conversation

abarciauskas-bgse
Copy link
Contributor

These are 2 minor changes required for 2 notebooks to work once we upgrade to pangeo-notebook base image 2024.01.23 (see NASA-IMPACT/veda-jh-environments#43 and of NASA-IMPACT/veda-jh-environments#42)

  1. timeseries-rioxarray-stackstac.ipynb: I cannot find in the CHANGELOG of stackstac what exactly changed but stackstac went from 0.4.4 to 0.5.0 in the pangeo-notebook image requirements and now requires the epsg to be explicitly passed because it is not set on the STAC items. This line is throwing the error but you can also see it in a0ae2a2

  2. downsample-zarr.ipynb: Is thworing a ValueError: 'epsg:6933' must be either a valid crs or an reference to a data.attr containing a valid crs: Projection must be defined as a EPSG code, proj4 string, WKT string, cartopy CRS, pyproj.Proj, or pyproj.CRS.. As far as I can tell the EPSG:6933 projection is not supported in version 0.9.1 of hvplot whereas it is supported in version 0.8.4. The best thing I can think to do is just specify to use hvplot==0.8.4 for this notebook to continue using that projection. (ref: SMAP L3 Radiometer Global Daily 36 km EASE-Grid Soil Moisture, Version 5)

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

PR Preview Action v1.4.7
🚀 Deployed preview to https://NASA-IMPACT.github.io/veda-docs/pr-preview/pr-119/
on branch gh-pages at 2024-02-15 15:41 UTC

@jsignell
Copy link
Collaborator

Included in #126

@jsignell jsignell closed this Mar 18, 2024
@jsignell
Copy link
Collaborator

Just got around to writing up 2. on the hvplot side: holoviz/hvplot#1329 and put up a PR 🤞🏻

@wildintellect wildintellect deleted the ab/upgrade-2024.01.23-fixes branch March 6, 2025 00:16
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.

2 participants