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

make_coupler_mosaic error in area_atm field. #265

Open
ngs333 opened this issue Dec 18, 2023 · 0 comments
Open

make_coupler_mosaic error in area_atm field. #265

ngs333 opened this issue Dec 18, 2023 · 0 comments
Labels

Comments

@ngs333
Copy link
Contributor

ngs333 commented Dec 18, 2023

make_coupler_mosaic_parallel (and possibly make_coupler_mosaic) is not correctly calculating or storing the area_atm field. The other fields (including area_lnd) seem to be correct. This is for at least for when the input is that of Test04.
Here are some additional observations and questions:
a) running with different number of mpi ranks yields different results for that field.
b) Test04 is not always detecting such failures when it is run.
c) Values for the field are often non-sensical, e.g. from e+220 to e-318, numerous are exactly 0. Some sections appearing random.
d) It may be interesting to know when this issue first appeared in the baseline.
f) It may be useful to know the consequences of calculating this particular field (area_atm) incorrectly.
g) the app does print some warnings, and these should be noted.

To Reproduce
Essentially run the tail end of Test04 (the calls to make_coupler_mosaic_parallel) with baseline code on
Analysis, and preferably with code compiled with the canonical build method.
Then compare with nccmp the land_mask_tileX.nc files in the /parallel directory and the one above it.
Note the discrepancies. Also run the serial version and compare the resultant files to those other .nc files produced by the serial program.

Expected behavior
There should be no difference in the land_mask_tileX.nc files.

System Environment
This can be observed on Analysis, building with the current canonical method (using the site-config, env.sh file, which currently loading for gcc 11)

@ngs333 ngs333 added the bug label Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant