Skip to content

Refresh hfradar notebook#318

Merged
MathewBiddle merged 6 commits into
mainfrom
refresh_hfradar_notebook
May 4, 2026
Merged

Refresh hfradar notebook#318
MathewBiddle merged 6 commits into
mainfrom
refresh_hfradar_notebook

Conversation

@ocefpaf
Copy link
Copy Markdown
Member

@ocefpaf ocefpaf commented May 2, 2026

Closes #316.

@MathewBiddle I tried to address the endpoint changes and the various options in the text. However, if we have some docs on those data, we could add a link to them as well.

PS: We do have 2 more archived notebooks that uses HF radar data. Should we try to un-archive those as well?

@ocefpaf ocefpaf requested a review from MathewBiddle May 2, 2026 12:41
@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented May 2, 2026

There are many failures here due to network issues, but some are due to the environment update and syntax/feature deprecations. I will address those in another PR.

@MathewBiddle
Copy link
Copy Markdown
Contributor

The notebook runs fine for me. I had to install dask, but that's because I'm on a new machine.

When you make the timeseries stick plot, would it be possible to pick one of the points in time to create a geospatial quiver plot to show some geospatial distribution? Or, would that require too much data over the wire?

PS: We do have 2 more archived notebooks that uses HF radar data. Should we try to un-archive those as well?

If we have other notebooks that could be updated, let's take a look and see what we can fix.

@MathewBiddle
Copy link
Copy Markdown
Contributor

When reviewing the last quiver plot, I didn't notice where the actual arrows were. Is that the geospatial extent of these data? I expected it to fill most of the map, not just in the New Orleans area.

Maybe showing the 2 km product or adjusting the color scale for the arrows will help accentuate them in the map?

@MathewBiddle
Copy link
Copy Markdown
Contributor

After digging a little bit (should have done that earlier) these are the only data available in the region and the coverage is what it is. So, I think this is good to merge for now.

Copy link
Copy Markdown
Contributor

@MathewBiddle MathewBiddle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@MathewBiddle MathewBiddle merged commit beb32bf into main May 4, 2026
8 of 12 checks passed
@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented May 4, 2026

When reviewing the last quiver plot, I didn't notice where the actual arrows were. Is that the geospatial extent of these data? I expected it to fill most of the map, not just in the New Orleans area.

Maybe showing the 2 km product or adjusting the color scale for the arrows will help accentuate them in the map?

We had a bit more in the original notebook. Not sure where the rest went. BTW, after looking at this data that I had the idea of reviving the other ones as well. Maybe those will be better to showcase hfrdata.

@ocefpaf ocefpaf deleted the refresh_hfradar_notebook branch May 4, 2026 12:31
@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented May 4, 2026

The notebook runs fine for me. I had to install dask, but that's because I'm on a new machine.

Dask is added in the environment in this PR. We did not had it before.

@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented May 4, 2026

The redirection is working on my end. Did you find any issue? (I'll update the URL in the next PR regardless. Trying also fix some of the failing notebooks while at it.)

@MathewBiddle
Copy link
Copy Markdown
Contributor

redirect works for me too. Wonder why Linkcheck is failing with it though

https://github.com/ioos/ioos_code_lab/actions/runs/25319067575/job/74223273024

@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented May 4, 2026

Connection timeouts can be transient, or maybe that site blocked requests from GitHub IPs. We'll know in the next PR when the direct URL will be used instead.

@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented May 4, 2026

BTW, the failure to publish the page is not related to that link. The Action that should push the pages did not run. Not sure why... Investigating.

@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented May 4, 2026

BTW, the failure to publish the page is not related to that link. The Action that should push the pages did not run. Not sure why... Investigating.

Oh! We moved to release based and we forgot. Minting a release in a moment.

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.

[New Notebook]: Access and visualize HFR data from NCEI

2 participants