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

Grounded berg in SIS2_bergs_cgrid #45

Closed
adcroft opened this issue Sep 5, 2014 · 0 comments
Closed

Grounded berg in SIS2_bergs_cgrid #45

adcroft opened this issue Sep 5, 2014 · 0 comments
Labels

Comments

@adcroft
Copy link
Collaborator

adcroft commented Sep 5, 2014

@nikizadehgfdl reported:

Even though the non-reproducibility across PE count changes seems to be fixed in dev/master for MOM6_SIS2_bergs_cgrid, the model crashes on day 12.5 with:

FATAL from PE 21: diamonds, thermodynamics: berg appears to have grounded!

This model used to run at least 20 days with the previous dev/master

Please extend the days=20 to verify this.

I have confirmed that the model crashes for the same reason at around this time using our interactive testing.

I also noticed two messages from the icebergs that are hidden when "verbose=.false.":

  1. bad lon: 218 4 210
  2. diamonds: net mass budget: recvd in= 4.5154404E+11 kg, rtrnd out= 2.2024064E+09 kg, Delta net mass= 4.4934164E+11 kg, error=-5.148E-01 nd
    The first means there are inconsistent grid data in, or on the edge of, the halo region. The second means there is a 50% error between in the mass budget as seen by SIS2 through modification of the calving field. For the corresponding SIS2_icebergs experiment (B-grid), this error is of order 1e-12. I speculate that we have a grid staggering issue somewhere! :)

Finally, the original grounded berg error indicated zero ice and ocean velocities but non-zero atmospheric winds. This event might be triggered by a new "situation" in which the C-grid ice-model permits non-zero winds on coast-lines. The bergs are meant to collide with the coast-line before landing inside a land cell so I'm not sure how this alone would lead to the above error.

@adcroft adcroft added the bug label Sep 5, 2014
@adcroft adcroft changed the title SIS2_bergs_cgrid Grounded berg in SIS2_bergs_cgrid Sep 5, 2014
Hallberg-NOAA added a commit that referenced this issue Sep 5, 2014
  Corrected bugs in which the u-stress was used in place of the v-stress in the
ice_bergs code with C-grid wind stresses.  This changes answers in the
SIS2_bergs_cgrid test case, and new reference solutions have been checked in.

  This closes #45 .
  This corresponds to NOAA-GFDL/SIS2@96a9a0b .
nikizadehgfdl pushed a commit to nikizadehgfdl/MOM6 that referenced this issue Oct 9, 2017
- Issue mom-ocean#45 pointed out that we had a bad runoff data set in our
  OM4 configuration: we should not interpolate runoff at runtime.
  Moreover, the file we are interpolating from is a 1-degree
  tri-polar grid but is treated as lat-lon data by data_override.
- This commit adds a preprocessing step that uses the runoff_regrid
  tool, following instructions sent by @amylangenhorst.
  - I have yet to actually test it in the model.
nikizadehgfdl pushed a commit to nikizadehgfdl/MOM6 that referenced this issue Oct 9, 2017
- As pointed out in issue mom-ocean#45, the runoff file in OM4_025 was
  somewhat bogus.
- Commit 7859155 the preprocessing steps to generate a 12 month
  climatology of runoff using fre-nctools, documented in the
  preprocessing directory.
- This commit only makes the file available but does not transition
  the default configuration to use it since I'd rather verify that
  the model runs stably before doing so.
- Closes mom-ocean#45.
- No answer changes (yet).
nikizadehgfdl pushed a commit to nikizadehgfdl/MOM6 that referenced this issue Oct 9, 2017
- In order to make progress on the issue mom-ocean#45 and make commit
  7859155 I added copies of make_coupler_mosaic.c and
  make_quick_mosaic.c to the preprocessing directory. Those
  files were subsequently found by path_names in the XML
  which caused compilation issues.
- Fortunately @Zhi-Liang has added the necessary patches to
  a branch which I am now cloning so we can do away with the
  local copies.
SMoorthi-emc pushed a commit to SMoorthi-emc/MOM6 that referenced this issue Dec 23, 2020
…L-20201215

Feature/update to gfdl 20201215
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant