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

[Merged by Bors] - feat: port Logic.Function.Basic #511

Closed
wants to merge 12 commits into from

Conversation

semorrison
Copy link
Contributor

@semorrison semorrison commented Oct 28, 2022

No description provided.

@semorrison
Copy link
Contributor Author

So far I've done the merge with the existing file, using previously ported proofs where the mathport proof is broken.

But I haven't attempted to

  • fix the remaining broken proofs
  • rename statements as appropriate (with #aligns)
  • edit for style

@semorrison semorrison added the help-wanted The author needs attention to resolve issues label Oct 28, 2022
@semorrison
Copy link
Contributor Author

The problem with congr has been fixed in Lean 4, but we can't move to nightly-2022-10-29 because of leanprover/lean4#1791.

pechersky and others added 4 commits October 30, 2022 00:06
Have to explicitly state `invFun` and `extend` via `@dite`,
zulip:
https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/Decidable.20of.20Exists.20changes.20.60if.60.20macro.20semantics

proofs for `Eq.mp`, `Eq.mpr`, `Eq.cast` don't see through to `Eq.rec` anymore
statements for `LeftInverse.eq_rec_eq` and `eq_rec_on_eq` use
explicit `@Eq.rec` for similar reason
Co-authored-by: Mario Carneiro <di.gama@gmail.com>
@semorrison semorrison added awaiting-review and removed help-wanted The author needs attention to resolve issues labels Oct 30, 2022
@semorrison semorrison changed the title feat: WIP port Logic.Function.Basic feat: port Logic.Function.Basic Oct 30, 2022
@gebner
Copy link
Member

gebner commented Oct 31, 2022

Depends on #522

@gebner gebner added blocked-by-other-PR This PR depends on another PR to Mathlib and removed awaiting-review labels Oct 31, 2022
@pechersky pechersky added awaiting-review and removed blocked-by-other-PR This PR depends on another PR to Mathlib labels Nov 1, 2022
@semorrison
Copy link
Contributor Author

bors merge

bors bot pushed a commit that referenced this pull request Nov 2, 2022
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Yakov Pechersky <ypechersky@treeline.bio>
@bors
Copy link

bors bot commented Nov 2, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Logic.Function.Basic [Merged by Bors] - feat: port Logic.Function.Basic Nov 2, 2022
@bors bors bot closed this Nov 2, 2022
@bors bors bot deleted the logic.function.basic branch November 2, 2022 22:36
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.

None yet

3 participants