Skip to content

remove type specification in the generic __normalize! #41045

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 5, 2021
Merged

remove type specification in the generic __normalize! #41045

merged 2 commits into from
Jun 5, 2021

Conversation

GiggleLiu
Copy link
Contributor

It will solve the following issue of differentiating normalize! with ForwardDiff. It does not make much sense to specify types in a "private" function.

julia> using ForwardDiff: Dual
julia> using LinearAlgebra

julia> normalize!(Dual.(randn(13), randn(13)))
ERROR: MethodError: no method matching __normalize!(::Vector{Dual{Nothing, Float64, 1}}, ::Dual{Nothing, Float64, 1})
Closest candidates are:
  __normalize!(::AbstractArray, ::AbstractFloat) at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/LinearAlgebra/src/generic.jl:1678
Stacktrace:
 [1] normalize!(a::Vector{Dual{Nothing, Float64, 1}}, p::Int64)
   @ LinearAlgebra /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/LinearAlgebra/src/generic.jl:1675
 [2] normalize!(a::Vector{Dual{Nothing, Float64, 1}})
   @ LinearAlgebra /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/LinearAlgebra/src/generic.jl:1674
 [3] top-level scope
   @ REPL[6]:1

imciner2
imciner2 reviewed Jun 2, 2021
View reviewed changes
@dkarrasch dkarrasch added the linear algebra Linear algebra label Jun 3, 2021
@dkarrasch dkarrasch merged commit 443a1b6 into JuliaLang:master Jun 5, 2021
shirodkara pushed a commit to shirodkara/julia that referenced this pull request Jun 9, 2021
@GiggleLiu
relax type specification in the generic __normalize! (JuliaLang#41045)
2b97fda
@KristofferC KristofferC mentioned this pull request Jun 13, 2021
Closed
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
@GiggleLiu @johanmon
relax type specification in the generic __normalize! (JuliaLang#41045)
6d93a53
@LilithHafner LilithHafner mentioned this pull request Oct 23, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andreasnoack andreasnoack andreasnoack left review comments

+1 more reviewer

@imciner2 imciner2 imciner2 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

linear algebra Linear algebra

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@GiggleLiu @andreasnoack @imciner2 @dkarrasch