Skip to content

Conversation

@zhaobin74
Copy link
Contributor

@zhaobin74 zhaobin74 commented Jul 11, 2025

NOTE

This PR depends on (or should be taken with) GEOS-ESM/GEOSgcm_GridComp#1136


This PR addresses GEOS-ESM/GEOSgcm_GridComp#1126 by

  • runoff is first processed offline and then fed by ExtData directly to Ocean GC

  • calving flux follows the same pathway

This PR conserves the total volume of runoff and calving from JRA55-DO

JRA55-DO_3_Runoff2001

@zhaobin74 zhaobin74 requested a review from a team as a code owner July 11, 2025 16:24
@zhaobin74 zhaobin74 added 0 diff dataocean Zero-diff for the dataocean model Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs labels Jul 11, 2025
@zhaobin74 zhaobin74 requested a review from amolod July 11, 2025 16:35
@zhaobin74 zhaobin74 changed the title improve JRA55-DO runoff distribution and add Ice calving flux v2 improve JRA55-DO runoff distribution and add Ice calving flux Jul 11, 2025
@zhaobin74 zhaobin74 marked this pull request as draft July 28, 2025 13:17
@zhaobin74
Copy link
Contributor Author

zhaobin74 commented Jul 28, 2025

Need to investigate the impact of solid discharge on sea ocean and sea ice.
Ready now.

@zhaobin74 zhaobin74 marked this pull request as ready for review July 28, 2025 15:47
@github-actions
Copy link

This PR is being prevented from merging because you have not added one of our required labels: 0 diff, 0 diff trivial, Non 0-diff, 0 diff structural, 0-diff trivial, Not 0-diff, 0-diff, automatic, 0-diff uncoupled, github_actions. Please add one so that the PR can be merged.

@mathomp4 mathomp4 removed the Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs label Nov 21, 2025
@mathomp4 mathomp4 added 0 diff coupled Zero-diff for the coupled model non 0-diff dataatm Non-zero-diff for dataatm Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs labels Nov 21, 2025
@github-actions
Copy link

This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff coupled Zero-diff for the coupled model 0 diff dataocean Zero-diff for the dataocean model Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs non 0-diff dataatm Non-zero-diff for dataatm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants