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

bugfix: properly import Base.cis into LinearAlgebra #42164

Merged
merged 1 commit into from
Sep 10, 2021

Conversation

olof3
Copy link
Contributor

@olof3 olof3 commented Sep 8, 2021

This fixes JuliaLang/LinearAlgebra.jl#865 (cis(::Union{RealHermSymComplexHerm,SymTridiagonal{<:Real}}) not properly exported from LinearAlgebra.)

For the changes to take effect, Revise.revise() seemed to be insufficient, but rebuilding julia with make worked. Perhaps it has something to do with the sysimage?

Also not sure if additional regression tests should be added, or if this kind of bug should be avoided by checking the coverage reports?

Certain methods, e.g.,
`cis(::Union{RealHermSymComplexHerm,SymTridiagonal{<:Real}})`,
were not imported properly.
@ararslan ararslan added backport 1.7 bugfix This change fixes an existing bug linear algebra Linear algebra labels Sep 10, 2021
@ararslan ararslan merged commit c4f0d8b into JuliaLang:master Sep 10, 2021
KristofferC pushed a commit that referenced this pull request Sep 11, 2021
Certain methods, e.g.,
`cis(::Union{RealHermSymComplexHerm,SymTridiagonal{<:Real}})`,
were not imported properly.

(cherry picked from commit c4f0d8b)
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
Certain methods, e.g.,
`cis(::Union{RealHermSymComplexHerm,SymTridiagonal{<:Real}})`,
were not imported properly.
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
Certain methods, e.g.,
`cis(::Union{RealHermSymComplexHerm,SymTridiagonal{<:Real}})`,
were not imported properly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This change fixes an existing bug linear algebra Linear algebra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Base.cis is not improted into LinearAlgebra
3 participants