-
Notifications
You must be signed in to change notification settings - Fork 48
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
Harmonize AllZip with All #138
Comments
Presumably, you mean adding a function like I agree that this would be good as well. In principle, I'd even like to just make |
Yes, such a function, making sure it can build the necessary evidence.
…On Tue, Apr 13, 2021, 2:25 AM Andres Löh ***@***.***> wrote:
Presumably, you mean adding a function like cpara_SList to AllZip?
I agree that this would be good as well.
In principle, I'd even like to just make AllZip c xs ys a synonym for All
(Uncurry c) (Zip xs ys) (which would explain the name choice), but I
currently don't see a way to do that while retaining the SameShapeAs
trick which improves inference.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#138 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAOOF7PMLSQ4II3HRFQ3S4TTIPPVFANCNFSM42RSQCFQ>
.
|
Also add a cpara-like function for AllZip as suggested in #138.
I've done this in #142. I haven't yet tested some other libs whether they compile ok with this change. |
All
comes with a method to use its constraints recursively;AllZip
does not. Presumably one of those designs is better and should be used for both.The text was updated successfully, but these errors were encountered: