Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix broken notebook #253

Merged
merged 4 commits into from
Feb 4, 2025
Merged

Fix broken notebook #253

merged 4 commits into from
Feb 4, 2025

Conversation

ocefpaf
Copy link
Member

@ocefpaf ocefpaf commented Feb 4, 2025

Alternative to #252.

In addition we fix a notebook that was failing due to server side restrictions.

@MathewBiddle
Copy link
Contributor

Looks like ioos_qc docs are down https://ioos.github.io/ioos_qc/
Causing linkrot to fail.

@ocefpaf
Copy link
Member Author

ocefpaf commented Feb 4, 2025

Looks like ioos_qc docs are down https://ioos.github.io/ioos_qc/ Causing linkrot to fail.

Yep. I'm working on it but I'm facing a cascade of erros and ended up in pyoceans/pocean-core#132... Will get there as soon as I can build ioos-qc docs again with latest pocean-core.

@ocefpaf
Copy link
Member Author

ocefpaf commented Feb 4, 2025

It should be OK now. We do have an odd redirect:

https://wiki.earthdata.nasa.gov/display/NASAISO/ISO+19115-2 - with Found to [https://urs.earthdata.nasa.gov/oauth/authorize?](https://urs.earthdata.nasa.gov/oauth/authorize?client_id=u1o_8zOuolqNjuLQz6wzYQ&response_type=code&redirect_uri=https%3A%2F%2Fwiki.earthdata.nasa.gov%2Fplugins%2Fservlet%2FConfluenceOAuth2&state=%2Fpages%2Fviewpage.action%3FspaceKey%3DNASAISO%26title%3DISO%2B19115-2)

It seems that the NASA resource now requries a login. The other failure is https://www.ncei.noaa.gov/metadata/geoportal , hopefully transiet.

TL;DR this PR is ready to go.

@ocefpaf ocefpaf requested a review from MathewBiddle February 4, 2025 19:33
@MathewBiddle
Copy link
Contributor

https://www.ncei.noaa.gov/metadata/geoportal is transient. I think they might be moving to another service. See #226

@MathewBiddle MathewBiddle merged commit f05b107 into ioos:main Feb 4, 2025
10 of 12 checks passed
@ocefpaf ocefpaf deleted the fix_broken_notebook branch February 4, 2025 19:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants