-
Notifications
You must be signed in to change notification settings - Fork 297
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] - refactor(ring_theory/ideal/*, ring_theory/jacobson): use comm_semiring
instead of comm_ring
for ideals
#5954
Closed
Closed
Changes from 13 commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
af91307
start using `comm_semiring` in `ideal`
adomani 5c99415
fix algebra/ring_quot
adomani e1c73cd
fix ring_theory/jacobson_ideal
adomani b796475
remove temporary file
adomani a432356
fix ring_theory/jacobson
adomani 3f9e0c7
Merge branch 'master' of github.com:leanprover-community/mathlib into…
adomani 9c04b60
Merge branch 'master' of github.com:leanprover-community/mathlib into…
adomani da02236
cleaning up
adomani d198c17
add module doc-strings
adomani d8bd499
move three lemmas to ring_theory/ideal/over
adomani e09e0d6
Update src/algebra/ring_quot.lean
adomani da22901
Update src/ring_theory/ideal/over.lean
adomani 26b66af
Update src/ring_theory/ideal/basic.lean
adomani 0360340
Update src/ring_theory/ideal/basic.lean
adomani 3271184
Update src/ring_theory/ideal/basic.lean
adomani b0d1dcd
Apply Bryan's suggestions from code review
adomani File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
At a glance, I'm surprised this isn't true for semirings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eric, you are right: below is a proof (the first one that I found) of the "same" result for a submodule of
R
, whenR
is asemiring
.However, at the moment,
ideal
is only defined for acomm_semiring
. In fact, I think that what you suggest should be a separate PR, in case it is not already there.More generally, we should have a "theory" of cyclic modules, i.e. modules generated by a single element, where this result would fit nicely, replacing
1 : R
by a generating element of the module.