Skip to content
This repository has been archived by the owner on Oct 8, 2024. It is now read-only.

Non-existent adoption/usage in the wild #24

Closed
pemrouz opened this issue Jul 23, 2019 · 3 comments
Closed

Non-existent adoption/usage in the wild #24

pemrouz opened this issue Jul 23, 2019 · 3 comments

Comments

@pemrouz
Copy link

pemrouz commented Jul 23, 2019

Whilst I think there is some value in doing this, it's a little bit concerning that usage of this approach of having transforms as iterators is effectively zero (itertools). To quote @domenic, do we need to get high adoption first? If not, at least explore/reflect on why the wider community has explored and decided not to take this approach?:

I think a crucial part of getting stage 2 (i.e., commitment that this should be in the language) is signs that this method gets high adoption, comparable to existing libraries, and that TC39 isn't pushing through something that the community is uninterested in. So I think it should be released during stage 1, and high adoption for the library will be a stage 2 prerequisite.

@domenic
Copy link
Member

domenic commented Jul 23, 2019

I think https://www.npmjs.com/package/lodash is pretty good evidence that the community is interested. (There, you do _(iterator).map(...) instead of iterator.map(...), because lodash wisely chose not to monkeypatch the built-in Iterator.prototype. But the idea is the same.)

@pemrouz
Copy link
Author

pemrouz commented Jul 23, 2019

That's a very vague connection. Being realistic and quantifying the overlap, this proposal doesn't cover 1% or 0.1% of the functionality of lodash. Even looking at the documentation for map, it doesn't cover a fraction of the same use cases. It also isn't designed around iterators. As debated in #8, lodash works on iterables - and even more powerfully non-iterables (object-to-object transformations). This proposal isn't to standardise lodash or able to effectively replace it to claim it as relevant prior art.

@ExE-Boss
Copy link
Contributor

ExE-Boss commented Aug 13, 2019

Java has this since version 8 in the form of streams.

@devsnek devsnek closed this as completed Oct 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants