Tim Spence

Results 56 comments of Tim Spence

> I listed a couple of minor nits, but I think any performance improvement is worth merging, especially around traverse since it is such a key function. Thanks! Do you...

@valencik you foolishly volunteered yourself on Discord to run the benchmarks as well 😆 If you have the chance, I would massively appreciate another set of data to confirm that...

@valencik absolute legend ❤️ Couple of points: 1. I'm dumb and not thinking straight 😅 I was thinking the `Map` implementation would be dependent on either `Chain` or `Vector` and...

@valencik sorry to bother you. Just tagging you in case you missed my previous comment

> > 2. `map` and `filter` should be unchanged by this so all of the results apart from `traverseError` are expected, right? It's certainly not obvious to me why `traverseError`...

Argh @valencik I've just realized the current HEAD uses `Vector` rather than `Chain`. From what I saw, `Chain` seemed to be better overall. Shall I switch it back? I think...