pcmdi_metrics icon indicating copy to clipboard operation
pcmdi_metrics copied to clipboard

Update central arctic domain

Open acordonez opened this issue 10 months ago • 3 comments

This PR expands the central arctic domain to remove the masking for the pole hole. On closer reading of the Ivanova (2016) paper, the pole hole was only masked for the ice edge analysis but not for the total ice area analysis.

acordonez avatar Apr 03 '24 17:04 acordonez

@acordonez Thank you for the PR. If I understood correctly, before this PR, the sea-ice metric removes the pole hole by masking it out because there is no observation. With this PR, the metric will include the sea-ice area in the pole hole. Is that right? Is there any chance that this PR could make the model looking like overestimating the sea-ice extent in case model does good job, because the model has data over the pole hole (that is counted for sea-ice area) while observation does not (thus not counted for sea-ice area)?

I think it is okay to improve metrics beyond what Ivanova (2016) did. I am tagging @gleckler1 and @durack1 for their insights.

lee1043 avatar Apr 03 '24 17:04 lee1043

@lee1043 That's correct. The OSI-SAF data we're using does have data over the hole, not sure how that's accomplished.

acordonez avatar Apr 03 '24 17:04 acordonez

@lee1043 Also looking at this code we've been including the sea ice data over the pole in the whole Arctic metrics, but not the central arctic metrics. If we do decide to mask the hole then I need to update the total Arctic region.

acordonez avatar Apr 03 '24 17:04 acordonez

@acordonez Do you think this exclusion of pole hole can be set as optional capability with a parameter accompanying with?

lee1043 avatar Apr 25 '24 19:04 lee1043

Note: pole option added by #1087

@acordonez please feel free to merge

lee1043 avatar Apr 30 '24 15:04 lee1043