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

Fixed handling of non-generator iterators #72

Merged
merged 1 commit into from
Dec 15, 2024
Merged

Fixed handling of non-generator iterators #72

merged 1 commit into from
Dec 15, 2024

Conversation

pelme
Copy link
Owner

@pelme pelme commented Dec 15, 2024

Previously there were checks for generators specifically and not iterators in general. We also support one-off iterators that are not based on generators such as itertools.chain().

This regression was introduced in #56.

Based on #71.

@pelme pelme force-pushed the push-twmxtqtpotzu branch 2 times, most recently from aec53b0 to b391be8 Compare December 15, 2024 09:54
@pelme
Copy link
Owner Author

pelme commented Dec 15, 2024

@astynax can you have a look and see if this solves your original problems?

@pelme pelme force-pushed the push-twmxtqtpotzu branch 8 times, most recently from 5c1e27d to e91370e Compare December 15, 2024 13:32
@pelme pelme changed the title Make possible to use one-shot iterators as children nodes. Fixed handling of non-generator iterators Dec 15, 2024
@pelme pelme force-pushed the push-twmxtqtpotzu branch 2 times, most recently from fb30eb6 to 142926d Compare December 15, 2024 13:51
@astynax
Copy link

astynax commented Dec 15, 2024

@astynax can you have a look and see if this solves your original problems?

Hi, @pelme ! Yes, now my code with a couple of chains works!

@pelme pelme force-pushed the push-twmxtqtpotzu branch from 142926d to 0e7372a Compare December 15, 2024 18:21
Previously there were checks for generators specifically and not iterators in
general. We also support one-off iterators that are not based on generators such
as itertools.chain().

This regression was introduced in #56.

Based on #71.
@pelme pelme force-pushed the push-twmxtqtpotzu branch from 0e7372a to f0d173d Compare December 15, 2024 18:24
@pelme pelme merged commit 11bf4ff into main Dec 15, 2024
14 checks passed
@pelme pelme deleted the push-twmxtqtpotzu branch December 15, 2024 18:28
@pelme
Copy link
Owner Author

pelme commented Dec 15, 2024

@astynax thanks for letting me know. will put out a new release soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants