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

[Merged by Bors] - feat(group_theory/monoid_localization): Order#18724

Closed
YaelDillies wants to merge 6 commits intomasterfrom
ordered_localization
Closed

[Merged by Bors] - feat(group_theory/monoid_localization): Order#18724
YaelDillies wants to merge 6 commits intomasterfrom
ordered_localization

Conversation

@YaelDillies
Copy link
Copy Markdown
Collaborator

Prove that every (linearly) ordered cancellative monoid can be embedded into a (linearly) ordered group, namely its Grothendieck group. Note that cancellativity is necessary since submonoids of a group are cancellative.


Open in Gitpod

@YaelDillies YaelDillies added awaiting-review The author would like community review of the PR t-algebra Algebra (groups, rings, fields etc) t-order Order hierarchy modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. labels Apr 3, 2023
@alexjbest
Copy link
Copy Markdown
Member

I guess you didn't quite do what the description says; can you add the order hom (or rel embedding or whatever it is) from the original monoid to the localisation that preserves the order when linear_ordered_cancel_comm_monoid?

@alexjbest
Copy link
Copy Markdown
Member

Should it be an ordered monoid embedding...? At least if for 1. Sorry for asking so much

@YaelDillies
Copy link
Copy Markdown
Collaborator Author

YaelDillies commented Apr 4, 2023

We don't have those (we don't even monoid embeddings). What other bundlings would you want (knowing that all the results are already there unbundled)?

@alexjbest
Copy link
Copy Markdown
Member

Yeah maybe its fine like this, I guess we could also have order_monoid_hom that is injective. But I suppose its not needed, maybe we could just add a TODO saying if an order_monoid_embedding is implemented this is an example

@alexjbest
Copy link
Copy Markdown
Member

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2023

🚀 Pull request has been placed on the maintainer queue by alexjbest.

Copy link
Copy Markdown
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.

bors d+

Thanks!

@bors
Copy link
Copy Markdown

bors bot commented Apr 19, 2023

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

@leanprover-community-bot-assistant leanprover-community-bot-assistant 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 Apr 19, 2023
@YaelDillies
Copy link
Copy Markdown
Collaborator Author

Thank you, that's a nice generalisation!

bors merge

bors bot pushed a commit that referenced this pull request Apr 19, 2023
Prove that every (linearly) ordered cancellative monoid can be embedded into a (linearly) ordered group, namely its Grothendieck group. Note that cancellativity is necessary since submonoids of a group are cancellative.



Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Apr 19, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Apr 19, 2023
Prove that every (linearly) ordered cancellative monoid can be embedded into a (linearly) ordered group, namely its Grothendieck group. Note that cancellativity is necessary since submonoids of a group are cancellative.



Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Apr 19, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(group_theory/monoid_localization): Order [Merged by Bors] - feat(group_theory/monoid_localization): Order Apr 19, 2023
@bors bors bot closed this Apr 19, 2023
@bors bors bot deleted the ordered_localization branch April 19, 2023 23:18
YaelDillies added a commit to leanprover-community/mathlib4 that referenced this pull request Apr 21, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 24, 2023
kim-em pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 10, 2023
hrmacbeth pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 10, 2023
hrmacbeth pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

delegated The PR author may merge after reviewing final suggestions. modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. t-algebra Algebra (groups, rings, fields etc) t-order Order hierarchy

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants