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 (RingTheory/PowerSeries): Add basic lemmas aiming at proving that power series over a field are a DVR #12160

Closed
wants to merge 11 commits into from

Conversation

faenuccio
Copy link
Collaborator

@faenuccio faenuccio commented Apr 15, 2024

Add some basic lemmas about (univariate) power series over fields and their inverses, aiming at proving that they form a DVR.

Co-authored-by: María Inés de Frutos Fernández @mariainesdff

@faenuccio faenuccio added WIP Work in progress t-algebra Algebra (groups, rings, fields etc) labels Apr 15, 2024
@faenuccio faenuccio added awaiting-review The author would like community review of the PR and removed WIP Work in progress labels Apr 16, 2024
Mathlib/RingTheory/PowerSeries/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/PowerSeries/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/PowerSeries/Inverse.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/PowerSeries/Inverse.lean Show resolved Hide resolved
Mathlib/RingTheory/PowerSeries/Inverse.lean Show resolved Hide resolved
Mathlib/RingTheory/PowerSeries/Inverse.lean Outdated Show resolved Hide resolved
Copy link
Member

@riccardobrasca riccardobrasca 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+

Mathlib/RingTheory/PowerSeries/Basic.lean Show resolved Hide resolved
@mathlib-bors
Copy link

mathlib-bors bot commented Apr 16, 2024

✌️ faenuccio 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-mathlib4-bot leanprover-community-mathlib4-bot added delegated and removed awaiting-review The author would like community review of the PR labels Apr 16, 2024
@faenuccio faenuccio added awaiting-review The author would like community review of the PR and removed delegated labels Apr 16, 2024
@faenuccio
Copy link
Collaborator Author

@riccardobrasca I have done several additions/modifications, you might want to have another round of reviews before I merge. In particular, I am shocked on how this line could possibly work (and solve all decidability issues).

@riccardobrasca
Copy link
Member

@riccardobrasca I have done several additions/modifications, you might want to have another round of reviews before I merge. In particular, I am shocked on how this line could possibly work (and solve all decidability issues).

I think this is more or less the same as opening Classical. I don't know if it is possible to have a "true" (I mean not given by choice, but by an actual algorithm) decidable equality on power series in certain cases, but if it is that line will probably create diamonds.

Mathlib/RingTheory/PowerSeries/Basic.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/PowerSeries/Inverse.lean Outdated Show resolved Hide resolved
Mathlib/RingTheory/PowerSeries/Inverse.lean Outdated Show resolved Hide resolved
faenuccio and others added 2 commits April 17, 2024 14:49
Co-authored-by: Riccardo Brasca <riccardo.brasca@gmail.com>
@faenuccio
Copy link
Collaborator Author

@riccardobrasca I have done several additions/modifications, you might want to have another round of reviews before I merge. In particular, I am shocked on how this line could possibly work (and solve all decidability issues).

I think this is more or less the same as opening Classical. I don't know if it is possible to have a "true" (I mean not given by choice, but by an actual algorithm) decidable equality on power series in certain cases, but if it is that line will probably create diamonds.

So what do you suggest me to do? To leave as it is, to open Classical or to manually add [Decidable (f =0)] (and [Decidable (0=0)] on one occasion) all the time?

@riccardobrasca
Copy link
Member

@riccardobrasca I have done several additions/modifications, you might want to have another round of reviews before I merge. In particular, I am shocked on how this line could possibly work (and solve all decidability issues).

I think this is more or less the same as opening Classical. I don't know if it is possible to have a "true" (I mean not given by choice, but by an actual algorithm) decidable equality on power series in certain cases, but if it is that line will probably create diamonds.

So what do you suggest me to do? To leave as it is, to open Classical or to manually add [Decidable (f =0)] (and [Decidable (0=0)] on one occasion) all the time?

Thinking more about it, it seems impossible to have an actual algorithm to decide equality, so opening Classical should be fine. Sorry for the noise.

Copy link
Collaborator Author

@faenuccio faenuccio left a comment

Choose a reason for hiding this comment

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

@riccardobrasca I have done several additions/modifications, you might want to have another round of reviews before I merge. In particular, I am shocked on how this line could possibly work (and solve all decidability issues).

I think this is more or less the same as opening Classical. I don't know if it is possible to have a "true" (I mean not given by choice, but by an actual algorithm) decidable equality on power series in certain cases, but if it is that line will probably create diamonds.

So what do you suggest me to do? To leave as it is, to open Classical or to manually add [Decidable (f =0)] (and [Decidable (0=0)] on one occasion) all the time?

Thinking more about it, it seems impossible to have an actual algorithm to decide equality, so opening Classical should be fine. Sorry for the noise.

Done! And thanks for all your comments.

@riccardobrasca
Copy link
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Apr 17, 2024
mathlib-bors bot pushed a commit that referenced this pull request Apr 17, 2024
…t power series over a field are a DVR (#12160)

Add some basic lemmas about (univariate) power series over fields and their inverses, aiming at proving that they form a DVR.

Co-authored-by: María Inés de Frutos Fernández @mariainesdff
@mathlib-bors
Copy link

mathlib-bors bot commented Apr 17, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat (RingTheory/PowerSeries): Add basic lemmas aiming at proving that power series over a field are a DVR [Merged by Bors] - feat (RingTheory/PowerSeries): Add basic lemmas aiming at proving that power series over a field are a DVR Apr 17, 2024
@mathlib-bors mathlib-bors bot closed this Apr 17, 2024
@mathlib-bors mathlib-bors bot deleted the fae_PR_PowerSeries branch April 17, 2024 17:11
uniwuni pushed a commit that referenced this pull request Apr 19, 2024
…t power series over a field are a DVR (#12160)

Add some basic lemmas about (univariate) power series over fields and their inverses, aiming at proving that they form a DVR.

Co-authored-by: María Inés de Frutos Fernández @mariainesdff
callesonne pushed a commit that referenced this pull request Apr 22, 2024
…t power series over a field are a DVR (#12160)

Add some basic lemmas about (univariate) power series over fields and their inverses, aiming at proving that they form a DVR.

Co-authored-by: María Inés de Frutos Fernández @mariainesdff
Jun2M pushed a commit that referenced this pull request Apr 24, 2024
…t power series over a field are a DVR (#12160)

Add some basic lemmas about (univariate) power series over fields and their inverses, aiming at proving that they form a DVR.

Co-authored-by: María Inés de Frutos Fernández @mariainesdff
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields etc)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants