search

JeffreySarnoff / NamedTupleTools.jl

some utilities for working with NamedTuples
MIT License
80 stars 11 forks source link

Infer select #27

Closed goretkin closed 4 years ago

goretkin commented 4 years ago

This is the interesting commit: https://github.com/JeffreySarnoff/NamedTupleTools.jl/commit/1bdfef63ecb0a4090cf4c95183cb621b2ef6991e

It seems like there are other functions that would benefit from a similar treatment, which is why I wrote #26 in general terms. Also the @inferred test might fail on Julia 1.0 [edit it passed].

JeffreySarnoff commented 4 years ago

What other functions would benefit from this treatment?

goretkin commented 4 years ago

I pushed a change for delete, which seemed to be an improvement. I thought there would be more functions, but these two were all I could find.

JeffreySarnoff commented 4 years ago

merging -- thank you! (I will add your name to the credits)

JeffreySarnoff commented 4 years ago

Thanks for the heads-up, no idea how that happened :) should be fixed now.