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

@trhille
Copy link
Collaborator

@trhille trhille commented May 6, 2022

Add test group for Koge Bugt S, Greenland. This will provide a useful way to work out issues related to narrow fjords on a small and affordable mesh. This contains a single mesh creation test case.

@trhille trhille added land ice in progress This PR is not ready for review or merging labels May 6, 2022
trhille added 3 commits May 11, 2022 08:21
Add test group for Koge Bugt S, Greenland. This contains a single
mesh creation test case. This test case works as desired, but documentation
still needs to be updated.
Add docs for Koge Bugt S mesh generation test case. Also change
default minimum grid spacing from 1km to 500m.
Change Koge Bugt S maximum grid spacing from 10 km to 4km to avoid
omitting the narrow up-glacier part of the catchment.
@trhille trhille force-pushed the add_koge_bugt_S_test_group branch from 84a74cb to 8e8996e Compare May 11, 2022 14:21
@trhille
Copy link
Collaborator Author

trhille commented May 11, 2022

Force-push from 84a74cb to 8e8996e was a rebase onto compass/master after merging #385.

@trhille trhille removed the in progress This PR is not ready for review or merging label May 11, 2022
@trhille trhille requested a review from matthewhoffman May 11, 2022 14:22
Copy link
Member

@matthewhoffman matthewhoffman left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for taking the extra time to add this experimental mesh to COMPASS.

@matthewhoffman matthewhoffman merged commit ba12ba2 into MPAS-Dev:master May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

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