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

Error on perlmutter with stealth feature log-law drag #6055

Closed
cbegeman opened this issue Nov 6, 2023 · 2 comments · Fixed by #6056
Closed

Error on perlmutter with stealth feature log-law drag #6055

cbegeman opened this issue Nov 6, 2023 · 2 comments · Fixed by #6056
Assignees
@cbegeman
Labels
bug mpas-ocean Stealth PR has feature which, if turned on, could change climate. fka FCC

Comments

@cbegeman
Copy link
Contributor

cbegeman commented Nov 6, 2023

The indices of kineticEnergyCell in the log-law drag function raise an error on perlmutter with gnu, mpich debug=on.

Migrated from MPAS-Dev/compass#722, compliments of @gcapodag.
@cbegeman cbegeman added bug mpas-ocean Stealth PR has feature which, if turned on, could change climate. fka FCC labels Nov 6, 2023
@cbegeman cbegeman self-assigned this Nov 6, 2023
This was referenced Nov 6, 2023
Merged
Closed
@gcapodag
Copy link
Contributor

gcapodag commented Nov 7, 2023

Hi @cbegeman sorry I could not get to this sooner. Do you need me to review it?

@cbegeman
Copy link
Contributor Author

cbegeman commented Nov 7, 2023

@gcapodag No rush on reviewing #6056 because we won't be getting it in until after the code freeze. But when it's convenient, it wouldn't hurt to have you try out the fix as well.

jonbob added a commit that referenced this issue Dec 14, 2023
@jonbob
Merge branch 'cbegeman/ocn-fix-loglaw-indices' into next (PR #6056)
75b8b5c 
Fix bug in stealth feature log-law drag

The indices of kineticEnergyCell in the log-law drag function raised an
error on perlmutter with gnu, mpich debug=on. This PR fixes the
indexing.

Fixes #6055

[BFB]
@jonbob jonbob closed this as completed in 0fd6f76 Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cbegeman cbegeman

Labels
bug mpas-ocean Stealth PR has feature which, if turned on, could change climate. fka FCC
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix bug in stealth feature log-law drag cbegeman/E3SM
2 participants
@gcapodag @cbegeman