search

JuliaMath / Calculus.jl

Calculus functions in Julia
Other
276 stars 76 forks source link

deprecate extension of Base.gradient #80

Closed mlubin closed 8 years ago

mlubin commented 8 years ago

This export is a common source of confusion for users of ForwardDiff, e.g. https://github.com/JuliaDiff/ForwardDiff.jl/issues/70. I don't think Calculus has a claim to extend Base's method any more so than ForwardDiff, so the least confusing solution is for neither to extend the method.

CC @jrevels

mlubin commented 8 years ago

CC @scidom

papamarkou commented 8 years ago

Good point, I agree; it may be better not to let Calculus extend the method as scope issues can be easily missed otherwise.