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.

Sign up for GitHub

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

Jump to bottom

Fixes crash if Langmuir number output is enabled but waves CS is not associated in KPP. #863

Merged
merged 1 commit into from
Oct 23, 2018
Merged

Fixes crash if Langmuir number output is enabled but waves CS is not associated in KPP. #863

merged 1 commit into from
Oct 23, 2018

Conversation

breichl
Copy link
Collaborator

@breichl breichl commented Oct 22, 2018

  • Previous commit added an optional output to KPP that breaks the code if the output is enabled but the waves module is not allocated.
  • Fixed by first checking if its present, then in nested loop checking if associated. If not associated, the output will not occur, but the model will not crash.

@Hallberg-NOAA
Copy link
Collaborator

This PR is being tested with https://gitlab.gfdl.noaa.gov/ogrp/MOM6/pipelines/6098

@Hallberg-NOAA Hallberg-NOAA merged commit cfcf555 into mom-ocean:dev/gfdl Oct 23, 2018
@adcroft adcroft mentioned this pull request Apr 22, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@breichl @Hallberg-NOAA