Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - feat(data/nat/modeq): add modeq and dvd lemmas from Apostol Chapter 5 #11787

Closed
wants to merge 21 commits into from

Conversation

stuart-presnell
Copy link
Collaborator

@stuart-presnell stuart-presnell commented Feb 2, 2022

Various lemmas about modeq from Chapter 5 of Apostol (1976) Introduction to Analytic Number Theory:

  • mul_left_iff and mul_right_iff: Apostol, Theorem 5.3
  • dvd_iff_of_modeq_of_dvd: Apostol, Theorem 5.5
  • gcd_eq_of_modeq: Apostol, Theorem 5.6
  • eq_of_modeq_of_abs_lt: Apostol, Theorem 5.7
  • modeq_cancel_left_div_gcd: Apostol, Theorem 5.4; plus other cancellation lemmas following from this.

Open in Gitpod

@stuart-presnell stuart-presnell marked this pull request as draft February 2, 2022 20:56
@vihdzp
Copy link
Collaborator

vihdzp commented Feb 3, 2022

Are we sure these theorems or anything similar to them aren't in mathlib already? They all seem pretty basic.

@stuart-presnell
Copy link
Collaborator Author

They do, but I couldn't resolve them with library_search.

src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
@stuart-presnell stuart-presnell marked this pull request as ready for review February 4, 2022 13:57
@stuart-presnell stuart-presnell added the awaiting-review The author would like community review of the PR label Feb 4, 2022
@stuart-presnell
Copy link
Collaborator Author

stuart-presnell commented Feb 4, 2022

@vihdzp I think I've addressed all your comments now. Thanks very much for your helpful suggestions, especially the suggested lemma names.

src/data/int/basic.lean Outdated Show resolved Hide resolved
Copy link
Collaborator

@kmill kmill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as I can tell, these are new lemmas, thanks.

src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
src/data/nat/modeq.lean Outdated Show resolved Hide resolved
@kmill kmill changed the title feat(data/nat/modeq): various lemmas from Apostol Chapter 5 feat(data/nat/modeq): add modeq and dvd lemmas from Apostol Chapter 5 Feb 9, 2022
@robertylewis
Copy link
Member

Thanks @stuart-presnell for the PR and @vihdzp for the detailed review!

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Feb 15, 2022
bors bot pushed a commit that referenced this pull request Feb 15, 2022
…#11787)

Various lemmas about `modeq` from Chapter 5 of Apostol (1976) Introduction to Analytic Number Theory:

* `mul_left_iff` and `mul_right_iff`: Apostol, Theorem 5.3
* `dvd_iff_of_modeq_of_dvd`: Apostol, Theorem 5.5
* `gcd_eq_of_modeq`: Apostol, Theorem 5.6
* `eq_of_modeq_of_abs_lt`: Apostol, Theorem 5.7
* `modeq_cancel_left_div_gcd`: Apostol, Theorem 5.4; plus other cancellation lemmas following from this.
@bors
Copy link

bors bot commented Feb 15, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(data/nat/modeq): add modeq and dvd lemmas from Apostol Chapter 5 [Merged by Bors] - feat(data/nat/modeq): add modeq and dvd lemmas from Apostol Chapter 5 Feb 15, 2022
@bors bors bot closed this Feb 15, 2022
@bors bors bot deleted the SP_modulo branch February 15, 2022 17:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants