Fix DGS test seed to avoid FMA-sensitive ill-conditioned matrix #1187
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The default seed generates a matrix for N=6, JTYPE=21 that causes DGGES to fail (INFO=9) when compiled without FMA instructions. Use a custom seed that avoids this specific ill-conditioned case while maintaining full test coverage.
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Fixes #1186
Description
The default random seed in the DGS test generates a matrix for N=6, JTYPE=21 that causes DGGES to fail when compiled without FMA instructions. This happens because 1-2 ULP differences from FMA vs non-FMA rounding accumulate through repeated DROT calls in DHGEQZ, leading to different execution paths in eigenvalue reordering.
This fix uses a custom seed that avoids the specific ill-conditioned matrix while maintaining full test coverage (all 26 matrix types, all dimensions).
Checklist