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

[ race ] Make Either be inside the racing primitive #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

buzden
Copy link
Owner

@buzden buzden commented Apr 1, 2022

This makes primitives for zipping and for racing looks very symmetric, in the form of C a -> C b -> C (Tambara a b) for particular tambaras of Pair and Either for zipping and racing respectively.

@buzden buzden added the enhancement New feature or request label Apr 1, 2022
@buzden
Copy link
Owner Author

buzden commented Apr 1, 2022

However, I doubt it's worth to be merged because it creates additional info (of the side using Either) and ignores it when it's not used (in <|>, most cases, I think) rather than adds this additional info only for when it's needed (as in #26). Thoughts of constant allocations and droppings tear apart my soul when I think about using Coop again with Arduino (although, the current design is not checked to be optimal according to allocations anyway).

This, by the way, causes different ordering of operations happened in the same moment of time, which is not beautiful, but I think is not really significant in real use.

@buzden buzden force-pushed the either-in-race-primitive branch from d51f383 to 460f60a Compare April 8, 2022 10:15
@buzden buzden force-pushed the either-in-race-primitive branch from 460f60a to be504e0 Compare February 13, 2023 08:03
@buzden buzden force-pushed the either-in-race-primitive branch from be504e0 to 7f8f5cc Compare November 1, 2023 22:33
@buzden buzden force-pushed the either-in-race-primitive branch from 7f8f5cc to 18095b7 Compare February 6, 2024 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant