JuliaCollections / Memoize.jl

@memoize macro for Julia
Other
175 stars 22 forks source link

Use get! instead of haskey/getindex/setindex! #52

Closed kmsquire closed 3 years ago

kmsquire commented 4 years ago
kmsquire commented 4 years ago

It turns out that this requires Julia 1.1 (and we're currently only testing on Julia 0.7 and 1.0).

I'll leave this open for now.

kmsquire commented 4 years ago

I just discovered https://github.com/marius311/Memoization.jl, which is a nice drop-in replacement for Memoize.jl, and which handles get! nicely.

@marius311, would you be interested in porting your version of memoization here (and become a maintainer)? Memoize.jl has been lacking active maintainership for a while, but it has the benefit of being in the JuliaCollections organization, and your code in Memoization.jl already seems to be a drop-in replacement.

(No worries if you'd rather keep your package separate.)