Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented Jul 24, 2024

We need to set all land-ice related variables to zero in critical passages:

  • landIceFracObserved (currently the case)
  • landIcePressureObserved
  • landIceDraftObserved
  • landIceGroundedFracObserved
  • landIceFloatingFracObserved
  • landIceThkObserved

We also need to set oceanFracObserved to one.

Checklist

  • Document (in a comment titled Testing in this PR) any testing that was used to verify the changes

We need to set all land-ice related variables to zero in critical
passages:

* landIceFracObserved (currently the case)
* landIcePressureObserved
* landIceDraftObserved
* landIceGroundedFracObserved
* landIceFloatingFracObserved
* landIceThkObserved

We also need to set `oceanFracObserved` to one.
@xylar
Copy link
Collaborator Author

xylar commented Jul 24, 2024

Testing

I tested this on Chrysalis along with #807 and examined the resulting landIce* variables and oceanFracObserved in ParaView. (In the process, I discovered that landIceThkObserved had been missed, and has been subsequently added here.)

@xylar xylar self-assigned this Jul 26, 2024
@xylar xylar added bug Something isn't working ocean labels Jul 26, 2024
@xylar xylar merged commit 643ae69 into MPAS-Dev:main Jul 26, 2024
@xylar xylar deleted the fix-land-ice-pressure-in-critical-passages branch July 26, 2024 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ocean

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.