Skip to content

[Merged by Bors] - chore: add instance from Nat.AtLeastTwo n to NeZero n#10964

Closed
timotree3 wants to merge 2 commits intomasterfrom
TCB/nezero-atleasttwo
Closed

[Merged by Bors] - chore: add instance from Nat.AtLeastTwo n to NeZero n#10964
timotree3 wants to merge 2 commits intomasterfrom
TCB/nezero-atleasttwo

Conversation

@timotree3
Copy link
Copy Markdown
Collaborator

@timotree3 timotree3 commented Feb 25, 2024

This PR cleans up some lemmas where we're taking named instance parameters [h : Nat.AtLeastTwo n] and manually proving inequalities like 0 < n from h by adding an instance of NeZero n from Nat.AtLeastTwo n as well as a couple of other helper lemmas.

This removes Nat.AtLeastTwo.ne_zero, as NeZero.ne_zero replaces it.


Split from #8006.

Open in Gitpod

@eric-wieser
Copy link
Copy Markdown
Member

!bench

@leanprover-bot
Copy link
Copy Markdown
Collaborator

Here are the benchmark results for commit 8bb99c1.
There were no significant changes against commit a62da66.

@eric-wieser
Copy link
Copy Markdown
Member

bors d+

after the change above

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 25, 2024

✌️ timotree3 can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). and removed awaiting-review labels Feb 25, 2024
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@eric-wieser
Copy link
Copy Markdown
Member

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Feb 25, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 25, 2024
This PR cleans up some lemmas where we're taking named instance parameters `[h : Nat.AtLeastTwo n]` and manually proving inequalities like `0 < n` from `h` by adding an instance of `NeZero n` from `Nat.AtLeastTwo n` as well as a couple of other helper lemmas.

This removes `Nat.AtLeastTwo.ne_zero`, as `NeZero.ne_zero` replaces it.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 25, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: add instance from Nat.AtLeastTwo n to NeZero n [Merged by Bors] - chore: add instance from Nat.AtLeastTwo n to NeZero n Feb 25, 2024
@mathlib-bors mathlib-bors bot closed this Feb 25, 2024
@mathlib-bors mathlib-bors bot deleted the TCB/nezero-atleasttwo branch February 25, 2024 21:06
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
This PR cleans up some lemmas where we're taking named instance parameters `[h : Nat.AtLeastTwo n]` and manually proving inequalities like `0 < n` from `h` by adding an instance of `NeZero n` from `Nat.AtLeastTwo n` as well as a couple of other helper lemmas.

This removes `Nat.AtLeastTwo.ne_zero`, as `NeZero.ne_zero` replaces it.
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
This PR cleans up some lemmas where we're taking named instance parameters `[h : Nat.AtLeastTwo n]` and manually proving inequalities like `0 < n` from `h` by adding an instance of `NeZero n` from `Nat.AtLeastTwo n` as well as a couple of other helper lemmas.

This removes `Nat.AtLeastTwo.ne_zero`, as `NeZero.ne_zero` replaces it.
xgenereux pushed a commit that referenced this pull request Apr 15, 2024
This PR cleans up some lemmas where we're taking named instance parameters `[h : Nat.AtLeastTwo n]` and manually proving inequalities like `0 < n` from `h` by adding an instance of `NeZero n` from `Nat.AtLeastTwo n` as well as a couple of other helper lemmas.

This removes `Nat.AtLeastTwo.ne_zero`, as `NeZero.ne_zero` replaces it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants