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

Sea ice reference data added to demo 0 downloading #1043

Merged
merged 9 commits into from
Feb 2, 2024

Conversation

lee1043
Copy link
Contributor

@lee1043 lee1043 commented Feb 2, 2024

EUMETSAT sea-ice reference data is added to demo 0 download, and note added to Demo 9 notebook

@lee1043 lee1043 self-assigned this Feb 2, 2024
@lee1043 lee1043 added the demo label Feb 2, 2024
@lee1043 lee1043 changed the base branch from main to 405_sic_ao February 2, 2024 02:15
Copy link
Collaborator

@acordonez acordonez left a comment

Choose a reason for hiding this comment

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

The notebook ran successfully but I was not able to download the files. Instead of netcdf files I get text files with this message. I'll let Peter know.

<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>403 Forbidden</title>
</head><body>
<h1>Forbidden</h1>
<p>You don't have permission to access /pss/pmpdata/misc_demo_data/ocn/ice_conc_nh_ease2-250_cdr-v3p0_198801-202012.nc
on this server.</p>
</body></html>

Copy link
Collaborator

@acordonez acordonez left a comment

Choose a reason for hiding this comment

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

The latest permissions update fixed this for me!

@lee1043
Copy link
Contributor Author

lee1043 commented Feb 2, 2024

@acordonez thank you for checking!

@lee1043 lee1043 merged commit b3d1ae6 into 405_sic_ao Feb 2, 2024
2 checks passed
@lee1043 lee1043 deleted the 405_sic_ao_lee1043_finaltweaks branch February 2, 2024 17:53
@lee1043
Copy link
Contributor Author

lee1043 commented Feb 2, 2024

I should have merged this directly to main... oops. I am opening a new PR to update main by merging 405_sic_ao for this change. Sorry for the confusion.

@lee1043 lee1043 added this to the 2.3.1 milestone Feb 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants