fix format string overflow in diag_integral test #1624
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The diag_integral test failing from the issue below ended up being from overflow due to the
xtime
variable's value having more digits before the decimal than the given width of 10 will allow. With this PR it'll check the count of digits before the decimal, and if its greater than ten it'll just output using the exponent notation instead.I cleaned up the
format_data_init
routine a bit by adjusting the indices used when it sets the data format string so it won't have additional spaces throughout. Also changed it's arguments to required (its private and only called with args) and removed some unused variables.Fixes #1623
How Has This Been Tested?
oneapi 2024.2 on the amd box.
Checklist:
make distcheck
passes