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.

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

SLAP issue with cce (cray compiler) #47

Open
jedwards4b opened this issue Nov 16, 2021 · 4 comments
Open

SLAP issue with cce (cray compiler) #47

jedwards4b opened this issue Nov 16, 2021 · 4 comments

Comments

@jedwards4b
Copy link
Contributor

A runtime error is generated in libglimmer-solve/SLAP/xersla.f with the cce/11.0.2 compiler.

A fix is to change dimension messg(nmessg) to be
character*(*) in the file xersla.f

@whlipscomb
Copy link
Contributor

@jedwards4b – As you may remember, the SLAP code isn't something we actively develop. It's legacy code that we carry around because it can be useful for testing new solvers in standalone CISM. We've talked about dropping it, in part because of issues like this.

For now, I'd suggest that we make this one fix and check it into the main branch. In the longer term, maybe we should modify the code so that CISM can be built in CESM without SLAP.

What do @billsacks and @Katetc think?

@billsacks
Copy link
Member

@whlipscomb sounds reasonable. See also #26 for some other minor SLAP issues that should be fixed, and #14 for thoughts on removing SLAP.

@whlipscomb
Copy link
Contributor

In the short term, I'll ask @Katetc to do a quick survey to find and fix the problematic lines of code in SLAP, and check the changes into the main branch. In the long term, I'll plan to remove SLAP from the CESM build. I'm sorry this remains an issue.

@Katetc
Copy link
Contributor

Katetc commented Nov 18, 2021

Sounds good to me! This should be pretty straight-forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants