-
Notifications
You must be signed in to change notification settings - Fork 15
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
Test fail on 1.2.0-rc #188
Comments
The deflation test failure seems to be because we get a different matrices from the gallery, so probably not entirely our fault 🎉 : Old:
New:
This seems to be due to JuliaLang/julia#30494 |
The julia developers in julia PR 30494 basically say that one cannot assume that the sequence of numbers generated by sprandn (as well as the other rand-functions) is not stable over julia versions. So we need to change the random examples, to not depend on random numbers. Also, we should never use a random starting vector in the tests (should be mostly be like that already). |
Decision on meeting 2019-08-28: Do a minimal version of a matrix (+sparse matrix) generator which is stable over versions. |
In the JuliaLang issue 27614 it is pretty clearly stated that the RNG-functionallity is not static. However, in the issue 32954 there is an argument to whether or not people need reproducibility. |
We could look into Middle Square Weyl Sequence RNG. It seems to not be secure, although that might not be a requirement for us. |
Initial implementation in aee82ec. |
Cool. Can you make a PR so it is easier to read when you want us to start looking at it? It doesn't have to be finished. Btw. I would recommend |
I think I am getting ready with the generation now. I might have gone overboard with the typeing. But since the inner most generator is hard-typed (from the algorithm) it somehow escaped out to the whole thing. |
Can you create a pr so we can have discussion there. It's easier to comment
on code etc.
|
Tests failing on the julia 1.2.0 release candidate:
Check when fixed:
The text was updated successfully, but these errors were encountered: