-
Notifications
You must be signed in to change notification settings - Fork 138
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
Mixed precision axis_utils unit tests - Let's try again #1172
Conversation
!* License along with FMS. If not, see <http://www.gnu.org/licenses/>. | ||
!*********************************************************************** | ||
|
||
program test_axis_utils |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs more documentation.
Something seems to be causing it to hang indefinitely while running the tests. Thought it might have been the outage, but github status shows everything as up now. Looks like this test was the issue when I ran it in the CI image (gcc 9):
Interestingly, the r4 version of the tranlon test is fine. Maybe this is related to the commented out lines in test? |
This is almost certainly due to |
@J-Lentz Nice, we can skip that test to get the CI passing for now since theres already a issue and PR for it. We'll just need to remember to re-enable it later on when the fix is in. If you set this in the script it should show the test as skipped and pass: SKIP_TESTS="test_axis_utils2.16" |
…re, and fms_mod (#1239) * feat: mixed precision axis_utils2 (#1104) * feat: mixed precision fms_mod (#1147) * feat: horiz interp mixed precision (#1067) * mixed precision sat_vapor_pressure (#1095) * feat: add mixed precision axis_utils unit tests (#1172) * fix: move type definitions to before first usage to fix nvhpc bug (#1187) * fix: change allocatable type for intel errors (#1221) Co-authored-by: Caitlyn McAllister <[email protected]> Co-authored-by: Jesse Lentz <[email protected]> Co-authored-by: MiKyung Lee <[email protected]>
…re, and fms_mod (#1239) (#1258) * feat: mixed precision axis_utils2 (#1104) * feat: mixed precision fms_mod (#1147) * feat: horiz interp mixed precision (#1067) * mixed precision sat_vapor_pressure (#1095) * feat: add mixed precision axis_utils unit tests (#1172) * fix: move type definitions to before first usage to fix nvhpc bug (#1187) * fix: change allocatable type for intel errors (#1221) Co-authored-by: Caitlyn McAllister <[email protected]> Co-authored-by: Jesse Lentz <[email protected]> Co-authored-by: MiKyung Lee <[email protected]>
Description
Unit tests have been implemented for all procedures in
axis_utils2
which deal with real numbers (as well asget_axis_cart
, which does not).Fully implemented tests
axis_edges
(already existed)get_axis_cart
lon_in_range
frac_index
nearest_index
tranlon
Skeletal tests (to be fully implemented at a later time)
interp_1d_1d
interp_1d_2d
interp_1d_3d
Not yet implemented
get_axis_modulo
get_axis_modulo_times
Tests which fail
tranlon
: Fails due to unexpected behavior when no elements are translated:istrt
is set to 0, when the expected value is 1. Shouldtranlon
be changed, or should the test be changed? See:tranlon
inaxis_utils2
returns erroneousistrt
value when no elements are moved #1156tranlon
: In addition, thetranlon
tests fail with gcc due to out-of-bound memory access: See:tranlon
inaxis_utils2
accesses and modifies out-of-bound memory #1137get_axis_cart
fails due to a string copy error when anaxis
orcartesian_axis
attribute is set to an unexpected value (lines 172-175 oftest_axis_utils.F90
). Shouldget_axis_cart
be changed, or should this be considered a user error and removed from the test?How has this been tested?
Tests compile and run on the AMD box with
gcc
andifort
.Checklist
make distcheck
passes