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(data/nat/gcd): add coprime_prod_left and coprime_prod_right #12268

Closed
wants to merge 2 commits into from

Conversation

nomeata
Copy link
Collaborator

@nomeata nomeata commented Feb 24, 2022


Open in Gitpod

Copy link
Member

@eric-wieser eric-wieser left a comment

Choose a reason for hiding this comment

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

LGTM, let's see if any other reviewers can think of a better home

@nomeata
Copy link
Collaborator Author

nomeata commented Feb 27, 2022

LGTM, let's see if any other reviewers can think of a better home

Any other reviewers with an opionion?


open_locale big_operators

lemma coprime_prod_left
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we not prove these for unique_factorization_monoids rather than just ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

They are specializions from these: https://leanprover-community.github.io/mathlib_docs/ring_theory/coprime/lemmas.html#is_coprime.prod_left because it's hard to use them at N. Having a single notion of coprime and a single set of lemmas that works everywhere would be great.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh I see, how unfortunate that nat.coprime has its own weird definition :-/

Copy link
Collaborator

Choose a reason for hiding this comment

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

(Even worse, nat.coprime is actually in the Lean library itself.)

@ocfnash
Copy link
Collaborator

ocfnash commented Mar 3, 2022

Thanks for these.

I think worth adding a reference to the more general versions https://leanprover-community.github.io/mathlib_docs/ring_theory/coprime/lemmas.html#is_coprime.prod_left (and right) in the doc strings but otherwise great.

bors d+

@bors
Copy link

bors bot commented Mar 3, 2022

✌️ nomeata 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 The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Mar 3, 2022
@nomeata
Copy link
Collaborator Author

nomeata commented Mar 4, 2022

Bors merge

@bors
Copy link

bors bot commented Mar 4, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(data/nat/gcd): add coprime_prod_left and coprime_prod_right [Merged by Bors] - feat(data/nat/gcd): add coprime_prod_left and coprime_prod_right Mar 4, 2022
@bors bors bot closed this Mar 4, 2022
@bors bors bot deleted the joachim/coprie_prod_left branch March 4, 2022 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated The PR author may merge after reviewing final suggestions.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants