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

Elements of Fin when converted to Nat are smaller than the Bound. #3407

Merged
merged 2 commits into from Oct 31, 2016

Conversation

xekoukou
Copy link
Contributor

@xekoukou xekoukou commented Sep 6, 2016

Are you interested in small proofs like these? I find myself proving things that seem to be of interest to the general public.

@Melvar
Copy link
Collaborator

Melvar commented Sep 6, 2016

My guess is that it’s the fact that your module is attempting to import itself that’s causing the error.

@jfdm
Copy link
Contributor

jfdm commented Sep 19, 2016

Rather than call the module Extra we should probably be consistent with similar purpose modules in base. For example compare the module (and modules under) Data.List.

@xekoukou
Copy link
Contributor Author

In the future, I expect proofs like this one to be moved to the main Fin module or somewhere else.
I think that we need a place where to dump proofs that we needed but couldn't find. That is why I put this on contrib so as to be considered experimental.

Then, after a while, someone needs to take the most useful proofs and categorize them.

Keep in mind that ,I think, one can define a minimum set of proofs, called axioms, and not need to pattern match on the Constructors of the Type anymore when proving things. But let us do this in a different time.

@ahmadsalim ahmadsalim merged commit 9dd3e00 into idris-lang:master Oct 31, 2016
@ahmadsalim
Copy link

Thanks for the PR!

@ahmadsalim
Copy link

@jfdm I just merged it since it was in contrib and if it gets important enough, it would be moved to a better named module anyway.

msmorgan pushed a commit to msmorgan/Idris-dev that referenced this pull request Aug 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants