Summon devs associated with fixed Underscore.js bugs. about lodash

[jdalton]

/cc @breakbild, @DTrejo, @gonsfx, @jeffv, @JulesAU

[jdalton]

/cc @cashlo, @fearphage, @kopious, @jou, @grignaak, @vkovalskiy

[jdalton]

/cc @lazylester, @mattyurka, @russplaysguitar, @iwyg, @chadhietala, @jeremyckahn, @taybin, @ironchefpython

[jeremyckahn]

Awesome, thanks for the update. Great work on this! I tried swapping Underscore for Lo-Dash in Rekapi some time back and a few tests broke, but it looks like Lo-Dash has grown quite a bit. I will give it another shot.

[jdalton]

@jeremyckahn

Great work on this!

Thanks!

I tried swapping Underscore for Lo-Dash in Rekapi some time back and a few tests broke, but it looks like Lo-Dash has grown quite a bit. I will give it another shot

While Lo-Dash is a drop-in replacement I have optimized and re-organized methods for common/tested/documented use. Some edge cases may slip through.

In this case the issue is _.pluck(this._actors, 'id') as Lo-Dash had optimized _.pluck to iterate over arrays-only (as thats the example and tested usage). I will re-evaluate the _.pluck optimization.

[jeremyckahn]

I will re-evaluate the _.pluck optimization.

Cool! I'll reconfigure Rekapi so that I can easily switch between Underscore and Lo-Dash for development. I think it's a good source of practical data points to test Lo-Dash against. This is a cool project and I'd love to help out!

[taybin]

Were these optimizations rejected from underscore? I'm a bit confused on why there's a rebranding of a fork without an attempt to get the improvements into the original project.

Although I am glad to see AMD support, which I know was declined by @jashkenas.

[jdalton]

@taybin

I'm a bit confused on why there's a rebranding of a fork without an attempt to get the improvements into the original project.

Lo-Dash started as a reverted pull request.

Check out my reply to "What makes Lo-Dash different from Underscore" – #28 (comment)

[taybin]

@jdalton Thanks. You might want to make the justification a little more prominent.

[jdalton]

@jeremyckahn

Ok, all Rekapi unit tests pass with the edge version of Lo-Dash.

@taybin

Thanks. You might want to make the justification a little more prominent.

I am prepping a third screencast to cover it.

[jeremyckahn]

@jdalton

Ok, all Rekapi unit tests pass with the edge version of Lo-Dash.

Sweet! Thanks for this, I am updating the Rekapi repo to easily switch between Lo-Dash and Underscore now.

[iwyg]

This is nice. I'll give this a try