chore(Algebra/Order): bound on |n / m|ₘ and |n - m|#37232
Conversation
PR summary 0f8bcdff02Import changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diff
You can run this locally as follows## summary with just the declaration names:
./scripts/pr_summary/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/pr_summary/declarations_diff.sh long <optional_commit>The doc-module for No changes to technical debt.You can run this locally as
|
|n / m|
|n / m||n / m|ₘ and |n - m|
| @[to_additive] lemma max_div_min_eq_mabs (a b : α) : max a b / min a b = |b / a|ₘ := by | ||
| rw [mabs_div_comm, max_div_min_eq_mabs'] | ||
|
|
||
| @[to_additive] lemma abs_div_lt_of_lt_lt {N M n m : α} (hn : 1 ≤ n) (hm : 1 ≤ m) (hnN : n < N) |
There was a problem hiding this comment.
I was confused for a moment until I saw that this was mabs, not abs. Let's clarify the name:
| @[to_additive] lemma abs_div_lt_of_lt_lt {N M n m : α} (hn : 1 ≤ n) (hm : 1 ≤ m) (hnN : n < N) | |
| @[to_additive] lemma mabs_div_lt_of_lt_lt {N M n m : α} (hn : 1 ≤ n) (hm : 1 ≤ m) (hnN : n < N) |
There was a problem hiding this comment.
Unfortunately I don't think @GrigorenkoPV will have permission to accept the suggestion, since the PR was created by a bot.
|
✌️ mathlib-splicebot[bot] can now approve this pull request. To approve and merge a pull request, simply reply with |
|
@Vierkantor I'm guessing you didn't mean to delegate to the bot? |
Ah oops! Yes, I did not take into account how GitHub's interface factors into this... @GrigorenkoPV I suppose if you agree with the change then I'll apply it and merge this PR myself using maintainer powers. |
This PR was automatically created from PR #34722 by @GrigorenkoPV via a review comment by @Vierkantor.