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

adding new notebook for CKAN API query #221

Merged
merged 3 commits into from
Sep 18, 2024

Conversation

MathewBiddle
Copy link
Contributor

closes #47

@ocefpaf
Copy link
Member

ocefpaf commented Sep 17, 2024

You can remove the COLAB badge, jupyterbook will add that in as a dropown option for us.

@MathewBiddle
Copy link
Contributor Author

Do we need all that install stuff too?

@MathewBiddle
Copy link
Contributor Author

I will be presenting this notebook tomorrow around 1pm ET. Probably too soon to push it out live for that though.

@ocefpaf
Copy link
Member

ocefpaf commented Sep 18, 2024

I will be presenting this notebook tomorrow around 1pm ET.

Online? I'd love to see that.

updating environment files for ckanapi
@MathewBiddle
Copy link
Contributor Author

I feel pretty confident that this is ready to go. I needed to add ckanapi to the environment.

It takes a while to run because it's pulling in data from 422 datasets...

@ocefpaf LMK if this is good to merge.

@MathewBiddle
Copy link
Contributor Author

Looks like 3 of the tests are failing. I think it's because of broken urls...

@ocefpaf
Copy link
Member

ocefpaf commented Sep 18, 2024

Looks like 3 of the tests are failing. I think it's because of broken urls...

Yep. Nothing related to this PR. We are good to merge this one.

@MathewBiddle MathewBiddle merged commit be47784 into ioos:main Sep 18, 2024
7 of 11 checks passed
@MathewBiddle
Copy link
Contributor Author

MathewBiddle commented Sep 18, 2024

Blerg. Looks like a broken link breaks the build? https://github.com/ioos/ioos_code_lab/actions/runs/10926041512/job/30328984459

(content/code_gallery/data_access_notebooks/2017-09-09-hurricane_irma: line 20010) broken    https://tidesandcurrents.noaa.gov/quicklook/view.html?name=IRMA - 404 Client Error: Not Found for url: https://tidesandcurrents.noaa.gov/quicklook/view.html?name=IRMA

@ocefpaf
Copy link
Member

ocefpaf commented Sep 18, 2024

Blerg. Looks like a broken link breaks the build?

I'm working on that one hold on...

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.

Query IOOS CKAN API for specific free text variable
2 participants