-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
Some tests are failing randomly from time to time #279
Comments
I've begun looking into this. Could you clarify what should happen when random state is set, please? As an alternative, I wondered whether a simpler test could be written. The current one is difficult for me to follow. |
I've begun carefully refactoring the testMoorePenroseInverse test to enhance clarity (Four Elements of Simple Design). To effectively test the Moore Penrose Inverse computation, is it necessary to set random state, @olekscode ? Due to my complete ignorance I am unsure of why. Thank you. |
Another test that's failing is
|
Another test that's failing erratically:
|
|
@olekscode I think I managed to corner the input matrix that is responsible for the erratically failing test:
This fails for the same reason as highlighted during the CI run. Thus we have a deterministic test we can use to corner the failure much more. If you'd like to investigate with me, you can find this in Does the above matrix relate in some way to your hypothesis of not setting the random state. My investigate points to an array |
I confirm this random failure: PMNumericalMethodsTestCase |
I fixed this one and all related to random matrix and vectors: PolyMathOrg/vector-matrix@5fb38e9 |
This is probably because they don't set the random state.
For example:
The text was updated successfully, but these errors were encountered: