search

FluxML / Functors.jl

Parameterise all the things
https://fluxml.ai/Functors.jl/stable/
MIT License
118 stars 15 forks source link

Deprecate fmap(walk, f, x, ys...) for simpler runwalk function #64

Closed gaurav-arya closed 1 year ago

gaurav-arya commented 1 year ago

The new structure avoids the need for a recursive closure. Resolves #62

Other than the deprecation, this should not be breaking according to the public API (and tests pass unchanged), although it should definitely be checked in practice.

gaurav-arya commented 1 year ago

Ack, this is probably breaking too due to the change in how recurse is used.