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

Potentially wrong grid size in shr_dmodel_readgrid for multi-column scm #6570

Open
mwarusz opened this issue Aug 29, 2024 · 4 comments
Open
Assignees
Labels
bug SCM single column model

Comments

@mwarusz
Copy link
Member

mwarusz commented Aug 29, 2024

I am not familiar with this code at all, but this line looks suspect to me and it caused overflows in some of my DP-SCREAM runs:

gsize = abs(scm_nx*scm_nx*nzgo)

Shouldn't it be scm_nx * scm_ny * nzgo ?

@bogensch
Copy link
Contributor

Thanks for reporting. So that I can reproduce the error could you please provide some more information for a setup where you see this error. Such as machine used, case, number of elements in the x and y directions and as well domain size. Providing the exact script you used would be best.

@rljacob rljacob added the SCM single column model label Aug 31, 2024
@ambrad
Copy link
Member

ambrad commented Sep 12, 2024

Excellent find, @mwarusz! I'm investigating another issue and have come across this same line as a problem. @bogensch it's possible that the GNU issues you see on PM are in fact due to this line. I have a reliable reproducer on Chrysalis with Intel running and GNU failing, and I isolated the seg fault to an operation that relies on this line being correct. I.e., with the fix, the case now runs.

@bogensch
Copy link
Contributor

bogensch commented Sep 12, 2024

Yes thanks @mwarusz and @ambrad, I will test this fix first chance I get. I must admit, I read this issue incorrectly the first time and I see now the error is pretty obvious.

@mahf708 mahf708 added the bug label Sep 13, 2024
@bogensch
Copy link
Contributor

I can report that putting this fix in solves all the issues I was having when attempting large domain sizes with DPxx. thanks for reporting @mwarusz! I will issue a PR for this fix early next week.

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

No branches or pull requests

5 participants