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

Add traverseFactorableSumWithAdjust and networkHoldFactorableSum #212

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

Ericson2314
Copy link
Member

@Ericson2314 Ericson2314 commented Jul 27, 2018

They avoid the samples of factorDyn by embracing the widgetHold pattern.

I am 100% cargo culting names here.

they avoid the samples of `factorDyn` by embracing the `widgetHold`
pattern.

I am 100% cargo culting names here.
--
-- It's often the case that some adjustable computation fed with sum types only
-- needs incremental adjustments when the variant stays the same but data within
-- changes, and only need be done from scatch when the the variant changes. This
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'scratch' ?

--
-- The singleton ['sing'], eliminator, and 'Prism'' family together connect the
-- sum type ['sum'] with its variants. The singleton must be total in that there
-- must be a singleon singleton assigned to each variant. The 'Prism'' family
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'singleon' ?

@hSloan hSloan added follow up Follow up with requester to take some action opinionated Requires judgement from senior developer or developer community labels Jan 23, 2019
@ali-abrar ali-abrar requested a review from cgibbard March 21, 2019 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement follow up Follow up with requester to take some action non-breaking change opinionated Requires judgement from senior developer or developer community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants