-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
WIP: Set up CI etc #1
Conversation
ebfce8d
to
afc5fac
Compare
The tests are so horribly made that they take forever to run so I think we need to run this on dedicated hardware and in parallel. |
Co-authored-by: Fredrik Ekre <ekrefredrik@gmail.com>
Can we reduce the number of tests but without significantly losing coverage? Maybe a major exercise. I suppose the easiest thing is to do the excision with the same test suite (use buildkite the way we do with LBT tests) and then hopefully over time clean up the testsuite. |
That's the plan, just throw our buildkite hardware resources at it and run it in parallel to not block this on anything. |
This PR validates the input parameters to the Julia LAPACK wrappers, so that the error messages are more informative. On nightly ```julia julia> using LinearAlgebra julia> LAPACK.geev!('X', 'X', rand(2,2)) ** On entry to DGEEV parameter number 1 had an illegal value ERROR: ArgumentError: invalid argument #1 to LAPACK call ``` This PR ```julia julia> using LinearAlgebra julia> LAPACK.geev!('X', 'X', rand(2,2)) ERROR: ArgumentError: argument #1: jobvl must be one of ('N', 'V'), but 'X' was passed ``` Secondly, moved certain allocations (e.g. in `geevx`) below the validation checks, so that these only happen for valid parameter values. Thirdly, added `require_one_based_indexing` checks to functions where these were missing. (cherry picked from commit dcd1fb2148dd7affee9449e304a9b42e7feba1ce) (cherry picked from commit 0087a1177509a6e70ff122636b0f7b6c596ce102)
No description provided.