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

Update nsidc g02202 Sea Ice Concentration to Version 5 #3869

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

rbeucher
Copy link
Contributor

Description

Update NSIDC g02202 Sea Ice Concentration to Version 5
Add Northern Hemisphere


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

New or updated data reformatting script


To help with the number of pull requests:

@rbeucher rbeucher requested a review from a team as a code owner January 24, 2025 01:29
@rbeucher rbeucher requested a review from flicj191 January 24, 2025 01:29
@rbeucher rbeucher force-pushed the update_nsidc_g02202 branch from 9375780 to 90c6586 Compare January 30, 2025 04:02
@rbeucher rbeucher force-pushed the update_nsidc_g02202 branch from 90c6586 to 607c52a Compare January 30, 2025 04:51
Copy link
Contributor

@flicj191 flicj191 left a comment

Choose a reason for hiding this comment

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

Works well for me,
I get this error when I load iris/fileformats/_nc_load_rules/helpers.py:1642: IrisCfInvalidCoordParamWarning: One of "scale_factor_at_projection_origin" and "standard_parallel" is required. which I'm not sure about, but still loads.

@@ -441,7 +441,7 @@ A list of the datasets for which a CMORizers is available is provided in the fol
+------------------------------+------------------------------------------------------------------------------------------------------+------+-----------------+
| NSIDC-0116-[nh|sh] [#note4]_ | usi, vsi (day) | 3 | Python |
+------------------------------+------------------------------------------------------------------------------------------------------+------+-----------------+
| NSIDC-g02202-[sh] | siconc (SImon) | 3 | Python |
| NSIDC-g02202-[nh|sh] | siconc (SImon) | 3 | Python |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| NSIDC-g02202-[nh|sh] | siconc (SImon) | 3 | Python |
| NSIDC-G02202-[nh|sh] | siconc (SImon) | 3 | Python |

update to capital 'G' used in data commands

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.

Update CMORiser and Downloader for NSIDC-G02202 to Support Version 5 and Missing North Hemisphere Part
2 participants