Skip to content

Update landlab notebooks - #171

Merged
mdpiper merged 8 commits into
csdms:mainfrom
smmorey:update-landlab-notebooks
Aug 5, 2026
Merged

Update landlab notebooks#171
mdpiper merged 8 commits into
csdms:mainfrom
smmorey:update-landlab-notebooks

Conversation

@smmorey

@smmorey smmorey commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

replaced the 03 notebook
added bijou ascii file to the data folder

@mdpiper

mdpiper commented Aug 5, 2026

Copy link
Copy Markdown
Member

I ran the linters on this PR. Once the CI finishes, I'll merge it.

@mdpiper

mdpiper commented Aug 5, 2026

Copy link
Copy Markdown
Member

The notebook tests are failing because the file used in notebook 1, cell 10, isn't in Ivy:

RasterioIOError                           Traceback (most recent call last)
Cell In[10], line 1
----> 1 with rasterio.open("../../../../data/boulder39p5N-106W_NASADEM.tif") as src:
      2     dem = src.read(1)  # reads the first band as a 2D array
      3     dx = src.res[
      4         0

I'll turn off testing for these notebooks so that they can be merged and used for class tomorrow, but they should be fixed.

@mdpiper

mdpiper commented Aug 5, 2026

Copy link
Copy Markdown
Member

This time, the CI failed because of the OT API key ran out of tries. I think this PR is OK to merge.

@mdpiper
mdpiper merged commit 05902dd into csdms:main Aug 5, 2026
7 of 8 checks passed
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