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

abs(::Dual) defined twice #310

Closed
tpapp opened this issue Mar 24, 2018 · 2 comments · Fixed by #311
Closed

abs(::Dual) defined twice #310

tpapp opened this issue Mar 24, 2018 · 2 comments · Fixed by #311

Comments

@tpapp
Copy link
Contributor

tpapp commented Mar 24, 2018

On latest package master and Julia v0.6.2:

julia> using ForwardDiff
WARNING: Method definition abs(ForwardDiff.Dual{T, V, N} where N where V<:Real) in module ForwardDiff at /home/tamas/.julia/v0.6/ForwardDiff/src/dual.jl:321 overwritten at /home/tamas/.julia/v0.6/ForwardDiff/src/dual.jl:166.
WARNING: Method definition abs(ForwardDiff.Dual{T, V, N} where N where V<:Real) in module ForwardDiff at /home/tamas/.julia/v0.6/ForwardDiff/src/dual.jl:321 overwritten at /home/tamas/.julia/v0.6/ForwardDiff/src/dual.jl:166.

(other warnings elided)

@jrevels
Copy link
Member

jrevels commented Mar 26, 2018

I'm guessing this is because you have a DiffRules installation containing JuliaDiff/DiffRules.jl#11? I'll go ahead and tag that, and open a PR to bump the DiffRules lower bound here and remove the manual abs primitive definition.

@tpapp
Copy link
Contributor Author

tpapp commented Mar 26, 2018

Yes. Sorry for not providing the version information,

julia> Pkg.status("DiffRules")
 - DiffRules                     0.0.3+             master

julia> Pkg.status("ForwardDiff")
 - ForwardDiff                   0.7.3+             master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants