Skip to content

Add Foxe Basin Throughflow #146

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

Merged
merged 2 commits into from
Jul 10, 2020

Conversation

proteanplanet
Copy link

@proteanplanet proteanplanet commented Jul 10, 2020

This pull request is at the suggestion of @milenaveneziani to include the Foxe Basin Through Flow in addition to the Arctic ship routes for WC14 r03, and is upstream of the Arctic Shipping PR . Transect added in this PR is indicated in red at the bottom of this map:

E3SM_Foxe_Basin_Passages

and is also available in KML here:

Foxe_Basin_Through_Flow.kml.zip

Please include @milenaveneziani in the review list to check that this is what she was requesting.

@xylar xylar requested review from xylar and milenaveneziani July 10, 2020 07:11
Copy link
Collaborator

@xylar xylar left a comment

Choose a reason for hiding this comment

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

This looks great to me. I won't do separate testing of this change since testing of the previous 2 PRs on shipping routes was time consuming and I don't anticipate trouble but let me know if you would like to see a mesh and it's worth several more hours' work.

@proteanplanet
Copy link
Author

@xylar Thanks for the fast turnaround. From my side, no need to see a WC14 mesh with this change, since it's almost open in #145 anyhow. It's also pretty obvious in EC30to60 what the impact would be. So I'm fine with proceeding to WC14 r03, which I believe is the last revision for E3SM V2, pending the approval of @milenaveneziani to this PR.

@xylar xylar self-assigned this Jul 10, 2020
@xylar xylar changed the title Potential inclusion of Foxe Basin Through Flow Add Foxe Basin Throughflow Jul 10, 2020
@xylar
Copy link
Collaborator

xylar commented Jul 10, 2020

Testing

In the end, I felt like the time it would take to test this now and make sure it works as expected would potentially save quite a bit more hassle down the road.

EC30to60

With Foxe Throughflow and Shipping Routes:
EC30to60

Without either:
old_EC30to60

WC14to60

With Foxe Throughflow and Shipping Routes:
WC14

Without either:
old_WC14

@milenaveneziani
Copy link
Collaborator

oh I missed this PR entirely yesterday! Thanks @proteanplanet for putting this in and @xylar for testing.
Don't forget to rebase after the other one is merged in.

@proteanplanet
Copy link
Author

@xylar Thanks for testing. With this, and #145, we should be ready to move forward with WC14 r03.

@xylar
Copy link
Collaborator

xylar commented Jul 10, 2020

@milenaveneziani, thanks for reviewing. A rebase shouldn't be necessary. That's convenient for looking at the diffs in the PR but makes no difference for the final state of the repo.

@xylar xylar merged commit 062d4b3 into MPAS-Dev:master Jul 10, 2020
@proteanplanet proteanplanet deleted the Protean/Andrew/develop/foxebasin branch July 10, 2020 21:18
mark-petersen added a commit to MPAS-Dev/MPAS-Model that referenced this pull request Aug 17, 2020
New Mesh: WC14to60kmL60E3SMv2r03 #628

Design discussion Water Cycle mesh, 14km high resolution region:

MPAS North America and Arctic Focused Water Cycle mesh for E3SM version
2, with a focused 14-km resolution around North America and 60 vertical
levels

This iteration of the WC14 mesh is with shipping tracks
(MPAS-Dev/geometric_features#145) and the Foxe Basin Throughflow
(MPAS-Dev/geometric_features#146) open in the Arctic.
xylar pushed a commit to xylar/old_compass2 that referenced this pull request Oct 12, 2020
New Mesh: WC14to60kmL60E3SMv2r03 #628

Design discussion Water Cycle mesh, 14km high resolution region:

MPAS North America and Arctic Focused Water Cycle mesh for E3SM version
2, with a focused 14-km resolution around North America and 60 vertical
levels

This iteration of the WC14 mesh is with shipping tracks
(MPAS-Dev/geometric_features#145) and the Foxe Basin Throughflow
(MPAS-Dev/geometric_features#146) open in the Arctic.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants