Skip to content

Switch wisc meshes to z-star#582

Closed
xylar wants to merge 5 commits intoMPAS-Dev:mainfrom
xylar:switch_global_ocean_wisc_to_z_star
Closed

Switch wisc meshes to z-star#582
xylar wants to merge 5 commits intoMPAS-Dev:mainfrom
xylar:switch_global_ocean_wisc_to_z_star

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented Mar 29, 2023

Checklist

  • User's Guide has been updated
  • Developer's Guide has been updated
  • Documentation has been built locally and changes look as expected
  • Document (in a comment titled Testing in this PR) any testing that was used to verify the changes

@xylar xylar requested a review from cbegeman March 29, 2023 14:50
@xylar xylar self-assigned this Mar 29, 2023
@xylar xylar added enhancement New feature or request ocean labels Mar 29, 2023
Copy link
Collaborator

@cbegeman cbegeman left a comment

Choose a reason for hiding this comment

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

Looks consistent with what we talked about. Would you like me to run some tests?

@xylar
Copy link
Collaborator Author

xylar commented Mar 29, 2023

@cbegeman, that would be wonderful! If you can run the ECwISC30to60 mesh, init and performance tests, that would be a really great starting point.

@xylar
Copy link
Collaborator Author

xylar commented Mar 29, 2023

If that goes well, we would also want to run dynamic_adjustment. That'll be a more rigorous test. Then, we'd probably turn on melting and run the simulation step for longer.

@xylar
Copy link
Collaborator Author

xylar commented Mar 29, 2023

It could be that init fails. I can't recall if global_ocean in init mode is prepared for that coordinate. I hope so!

@cbegeman
Copy link
Collaborator

@xylar The model crashes (invalid state) at the second time step of ssh_adjustment when dt=30s

@xylar
Copy link
Collaborator Author

xylar commented Mar 31, 2023

Thanks. Not promising, it sounds like. Presumably, waves are big enough to evacuate thin water columns.

@cbegeman
Copy link
Collaborator

@xylar I just reran the case with wetting and drying turned on and it doesn't appear that thin layers are actually evacuated. The config_verify_not_dry functionality should have caught that.

@xylar
Copy link
Collaborator Author

xylar commented Apr 2, 2023

Thanks @cbegeman. I'll need to revisit this, then. I think I will prioritize other changes (like bathymetry, initial condition and vertical grid) first since they may affect this behavior.

@xylar xylar marked this pull request as draft April 2, 2023 09:22
@xylar xylar added the in progress This PR is not ready for review or merging label Apr 2, 2023
@xylar xylar force-pushed the switch_global_ocean_wisc_to_z_star branch from 01edf8f to b9cd8be Compare May 12, 2023 14:41
@xylar
Copy link
Collaborator Author

xylar commented Jun 1, 2023

I rebased this after various recent changes (e.g. to initial conditions and vertical coordinate) and it just worked! I was able to run mesh, init and performance_test on Chrysalis with the ECwISC30to60. I'm now running dynamic adjustment and will follow up with other meshes if all goes well.

@xylar xylar marked this pull request as ready for review June 1, 2023 15:11
@xylar xylar removed the in progress This PR is not ready for review or merging label Jun 1, 2023
@cbegeman
Copy link
Collaborator

cbegeman commented Jun 1, 2023

@xylar That's great news!

@xylar
Copy link
Collaborator Author

xylar commented Jun 1, 2023

It seems like some adjustments will be needed to dynamic adjustment (very meta) but no sense so far that it's an insurmountable problem.

@xylar xylar force-pushed the switch_global_ocean_wisc_to_z_star branch from b9cd8be to b73f3e5 Compare June 1, 2023 16:11
@xylar xylar force-pushed the switch_global_ocean_wisc_to_z_star branch from d3747ca to b71f5c1 Compare June 3, 2023 16:41
@xylar
Copy link
Collaborator Author

xylar commented Aug 14, 2023

I'm going to close this. I don't think we will be able to switch to the z-star vertical coordinate here on compass, and we should focus on doing that as part of the move to polaris this fall.

@xylar xylar closed this Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ocean

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants