Brutkey

Lynnesbian :bune_ylw:
@lynnesbian@fedi.lynnesbian.space
github feature proposal simulator

Bloplang Pull Request #48592: Add zip method to iterators gh_closed_1​gh_closed_2​gh_closed_3​

bean_red This is a proposal to add the zip method to iterators. This would allow you to join two iterators together, alternating between results from both. E.g. tens.zip(hundreds) would give [10, 100, 20, 200, 30].... This has been the most requested feature on our wishlist for seven months, and my initial implementation here provides a 12% speed up on the test_cases/thousands_of_types.blop. 👍👍 532 👎👎 12 🚀🚀 226 👀👀 114 😕😕 3

bot BlobBot says: Hi red_bean bean_red ! Thanks for opening a pull request to Bloplang! I've run your code, and found this: Maximum regression: 0.05%. Maximum improvement: 12.03%. I've assigned curly_brace curly_brace to approve your code. Don't forget to leave a star! 👍👍 221 🚀🚀 179 👀👀 12

bot BlopBot added the "BlopBot Seal of Approval" "No regressions" labels.

rct_guest I've never heard of "zip" before. Googling it, it seems to be a compression algorithm? Not sure what this has to do with iterators. I suggest naming it interleave instead. 👍👍 23 👎👎 34

bean_red zip seems to be pretty widely accepted. For example, Rust and Python call it zip. 👍👍 27 😕😕 1

green_is_my_pepper yeah i've never head of zip before. interleave is too long tho, how about weave 👍👍 54 👎👎 27

↕↕️ This comment was marked as off-topic.

ℹℹ️ curly_brace requested 1 change.

curly_brace This for loop could be better expressed as a map. 😕😕 1
acnl_smile How about spritz? In the idiolect of English spoken by my Discord server, spritz is very commonly used to refer to mixing things together. EDIT If you're going to laugh react, please leave a comment explaining why. 👍👍 5 😆😆 143

green_is_my_pepper wtf is a spritz 👎👎 5 😕😕 26

↕↕️ This comment was marked as off-topic.

↕↕️ This comment was marked as off-topic.

curly_brace Please don't leave "+1" comments. They pollute the thread and send useless emails. Use the react feature to add thumbs ups to the pull request if you want to express your approval. 👍👍 118 👎👎 4

gynecologist who else is here from reddit? 👎👎 122 😆😆 16 😕😕 33
[ Click to load 281 additional comments ] *

curly_brace That's a bug with your Linux distribution’s zip command, which has nothing to do with Bloplang. Open a thread on the Fedora forums to discuss that, although they'll probably tell you that Fedora 32 is EOL. 👍👍 66

rct_guest Okay, so we have 17 votes in favour of zip, 12 for weave, 5 for spritz, and somehow only 2 for interleave. Oh, and 1 each for sprinkle, intersperse_at_least_two_iterators_with_each_other, and sum. I'm beginning to suspect botting. 👍👍 7 😕😕 12

curly_brace Hey read bean bean_red - this pull request is out of sync with main. Could you rebase on top of the twelve thousand commits that have occurred since you opened this? Thanks! 👍👍 63 🚀🚀 2

gale whats stoping this from being merged? can we get an eta 👍👍 12 😕😕 2

rct_guest It's mostly a debate on the name to use. I'm not sure why people aren't voting for interleave. I've provided numerous arguments as to why it's the obvious option, and I've even linked a conversation with ChatGPT 4-o proving that it makes the most sense. Sigh. 👍👍 2 😕😕 70 😆😆 11

ℹℹ️ Pull request #53223 "Add interleave method to iterators" was marked as a duplicate of this one.

↕↕️ This comment was marked as off-topic.

🤖🤖 No activity detected for 90 days. 👎👎 121 😕😕 89

Stalebot 🤖🤖 marked this pull request as stale.

Conversation locked and limited to contributors.