Skip to content

Convert integer cast to round() for safety#4

Open
heshpdx wants to merge 1 commit intoANL-CESAR:masterfrom
heshpdx:master
Open

Convert integer cast to round() for safety#4
heshpdx wants to merge 1 commit intoANL-CESAR:masterfrom
heshpdx:master

Conversation

@heshpdx
Copy link

@heshpdx heshpdx commented Jan 4, 2023

Fix for a verification error on AMD EPYC system using a Cray compiler version 15. Building with -Ofast exposed the problem.
Using these inputs:

axial z-ray separation:            0.25
assemblies per axial sub-domain:   20
reactor height:                    400.00

On the system under test, the I->z_stacked was 79.999992 which was casted down to 79. Using round() fixed this so I->z_stacked computes to 80.

Fix for a verification error on AMD EPYC system using a Cray compiler version 15. Building with -Ofast exposed the problem.
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

Successfully merging this pull request may close these issues.

1 participant