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

fix \(A::HermOrSym{BigFloat}, b::AbstractVector) #22960

Merged
merged 2 commits into from
Jul 26, 2017
Merged

Conversation

fredrikekre
Copy link
Member

@fredrikekre fredrikekre commented Jul 25, 2017

@ararslan ararslan added bignums BigInt and BigFloat linear algebra Linear algebra labels Jul 25, 2017
@ararslan ararslan requested a review from kshyatt July 25, 2017 22:14
@fredrikekre
Copy link
Member Author

Also pushed a commit enabling more BigFloat tests for HermOrSym{BigFloat} instead of spamming with PR's.

@@ -297,24 +297,22 @@ end
Base.LinAlg.A_mul_B!(C,a,Symmetric(asym))
@test C ≈ a*asym

if eltya != BigFloat
tri_b = UpperTriangular(triu(rand(eltya, n, n)))
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kind of silly to not test this just because we cannot rand BigFloats :)

@fredrikekre
Copy link
Member Author

Timeout on Travis i686, #22552 on 32-bit AV.

@fredrikekre fredrikekre merged commit 0a37b3d into master Jul 26, 2017
@fredrikekre fredrikekre deleted the fe/solve-bigfloat branch July 26, 2017 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bignums BigInt and BigFloat linear algebra Linear algebra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants