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

feat(library/init/core): allow lists of proofs #533

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

eric-wieser
Copy link
Member

With upstream changes to mathlib, this would allow fintype some_prop, which would in turn allow proofs to be used to index finite things like matrices or summations.

With upstream changes to mathlib, this would allow `fintype some_prop`, which would in turn allow proofs to be used to index finite things like matrices or summations.
@gebner
Copy link
Member

gebner commented Feb 18, 2021

IIRC it was intentional that the type parameter of list and option is in Type instead of Sort, as this simplifies type inference. Have you tried compiling mathlib with this change?

@eric-wieser
Copy link
Member Author

I have not at any point tried to build lean itself locally, and was relying on CI to do that. Presumably once this build passes, I can create a mathlib branch that points to this branch, and have CI test that too?

@gebner
Copy link
Member

gebner commented Feb 18, 2021

You need to make a tag for the CI to upload a release. Please make the tag on your own fork.

@eric-wieser
Copy link
Member Author

eric-wieser commented Feb 18, 2021

Build is full of failures with trivial goals that simp isn't closing:

/home/runner/work/lean/lean/library/init/data/list/lemmas.lean:25:3: error: tactic failed, there are unsolved goals
state:
case list.cons
α : Type u,
t_hd : α,
t_tl : list α,
t_ih : t_tl ++ nil = t_tl
⊢ t_hd = t_hd ∧ t_tl = t_tl

I guess I'll have to try this locally...

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

Successfully merging this pull request may close these issues.

2 participants