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

Merge Data.Arity.Linear.Peano with Unsafe.INat #392

Open
tbagrel1 opened this issue Feb 22, 2022 · 2 comments
Open

Merge Data.Arity.Linear.Peano with Unsafe.INat #392

tbagrel1 opened this issue Feb 22, 2022 · 2 comments

Comments

@tbagrel1
Copy link
Member

The need for an alternative Nat definition arose in both Unsafe and Data.{V,Replicator}.Linear modules.

As a result, we created Peano and INat with the same definition and purpose. We should merge them.

@treeowl
Copy link
Collaborator

treeowl commented Apr 12, 2022

While we're at it, why does Arity produce a Nat rather than a Peano? I think we always convert the result to a Peano before we actually use it.

@aspiwack
Copy link
Member

I think that there was a reason. Or maybe it's simply because it was a quicker refactor. But either way, I don't remember. It may be worth trying and see what feels best.

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

No branches or pull requests

3 participants