Skip to content
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

Use 0.3.13+4 openblas binaries to fix #40279 #40343

Merged
merged 4 commits into from
Apr 4, 2021
Merged

Use 0.3.13+4 openblas binaries to fix #40279 #40343

merged 4 commits into from
Apr 4, 2021

Conversation

ViralBShah
Copy link
Member

@ViralBShah ViralBShah commented Apr 4, 2021

We should add the test for JuliaLang/LinearAlgebra.jl#833 to this PR.

Also some doctest failures. I think those doctests need updating to match the new binaries (looks harmless - some small changes in a couple of calculations, and sign changes in some eigenvector tests).

@pablosanjose Would it be possible for you to provide the test for QZ and the doctest updates?

@ViralBShah ViralBShah added the linear algebra Linear algebra label Apr 4, 2021
@ViralBShah ViralBShah added the backport 1.6 Change should be backported to release-1.6 label Apr 4, 2021
@pablosanjose
Copy link
Contributor

I can confirm this fixes JuliaLang/LinearAlgebra.jl#833
I will add some tests

@pablosanjose
Copy link
Contributor

Created #40346 with tests. Is this the right way to contribute to an open PR? Or is there a way to contribute without opening a new PR?

* tests and doctest fixes

* changed a doctest to have nicer output
@ViralBShah ViralBShah merged commit 58fba2b into master Apr 4, 2021
@ViralBShah ViralBShah deleted the vs/40279 branch April 4, 2021 23:29
@KristofferC KristofferC mentioned this pull request Apr 6, 2021
33 tasks
KristofferC pushed a commit that referenced this pull request Apr 6, 2021
* Use 0.3.13+4 openblas binaries to fix #40279
* Add tests and doctest fixes for #40343 (#40346)
* Update checksums
* changed a doctest to have nicer output

Co-authored-by: Pablo San-Jose <[email protected]>
(cherry picked from commit 58fba2b)
@KristofferC
Copy link
Member

Dropping label until JuliaLang/LinearAlgebra.jl#834 is resolved.

@KristofferC KristofferC removed the backport 1.6 Change should be backported to release-1.6 label Apr 6, 2021
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
* Use 0.3.13+4 openblas binaries to fix #40279
* Add tests and doctest fixes for JuliaLang#40343 (JuliaLang#40346)
* Update checksums
* changed a doctest to have nicer output

Co-authored-by: Pablo San-Jose <[email protected]>
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
* Use 0.3.13+4 openblas binaries to fix #40279
* Add tests and doctest fixes for JuliaLang#40343 (JuliaLang#40346)
* Update checksums
* changed a doctest to have nicer output

Co-authored-by: Pablo San-Jose <[email protected]>
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
* Use 0.3.13+4 openblas binaries to fix #40279
* Add tests and doctest fixes for JuliaLang#40343 (JuliaLang#40346)
* Update checksums
* changed a doctest to have nicer output

Co-authored-by: Pablo San-Jose <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linear algebra Linear algebra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants