Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sequence-based implementation of combining algorithms in addition to variadic ones #226

Open
MaxDesiatov opened this issue Nov 2, 2022 · 2 comments
Labels
enhancement New feature or request v1.1 Post-1.0 work

Comments

@MaxDesiatov
Copy link
Member

MaxDesiatov commented Nov 2, 2022

Existing implementations of combineLatest , zip , merge are great when the total number of sequences is known at compile time (2 or 3 without variadic generics). In some scenarios, these sequences have same element types, but the total number of sequences is computed at run time. Existing implementations won't work here, and I think the library would benefit from implementations that take a Sequence (or some other container protocol) of AsyncSequences that have the same element type to cover these use cases.

@MaxDesiatov MaxDesiatov changed the title Add Sequence-based implementation of combining algorithms in addition to variadic ones Sequence-based implementation of combining algorithms in addition to variadic ones Nov 2, 2022
@FranzBusch
Copy link
Member

Thanks for opening this @MaxDesiatov. This is something we can look into after the 1.0 release!

@FranzBusch FranzBusch added the v1.1 Post-1.0 work label Nov 2, 2022
@FranzBusch FranzBusch added the enhancement New feature or request label Sep 21, 2023
@clive819
Copy link

This is my attempt to make combineLatest support variadic parameters.

https://github.com/apple/swift-async-algorithms/pull/322/files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request v1.1 Post-1.0 work
Projects
None yet
Development

No branches or pull requests

3 participants