You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey Kevin, I took a crack at providing a more optimized System.Linq here.
It was kind of in the opposite direction to you, I was not minimizing garbage, but rather changing the implementation of enumerable consumers to be push fed rather than pull, which in the implementation meant that I was creating more garbage, not less, but it is faster in pretty much all but the plain vanilla IEnumerable being consumed directly.
Anyway, understandably for such a large change I'm struggling to find traction within the Microsoft camp to accept this as a PR, so I'm thinking of spinning it off into it's own repo and nuget package (it's somewhere in the order of 75%-90% done I guess - main feature still to migrate is OrderBy, but need probably better implementations of some other things that I have done).
Wondering if you would be interested in collaboration/review/whatever.
The text was updated successfully, but these errors were encountered:
@kevin-montrose
Hey Kevin, I took a crack at providing a more optimized System.Linq here.
It was kind of in the opposite direction to you, I was not minimizing garbage, but rather changing the implementation of enumerable consumers to be push fed rather than pull, which in the implementation meant that I was creating more garbage, not less, but it is faster in pretty much all but the plain vanilla IEnumerable being consumed directly.
Anyway, understandably for such a large change I'm struggling to find traction within the Microsoft camp to accept this as a PR, so I'm thinking of spinning it off into it's own repo and nuget package (it's somewhere in the order of 75%-90% done I guess - main feature still to migrate is OrderBy, but need probably better implementations of some other things that I have done).
Wondering if you would be interested in collaboration/review/whatever.
The text was updated successfully, but these errors were encountered: