Skip to content

[Merged by Bors] - feat(Algebra/Ring): generalise and extend material about sums of squares and semireal rings#16094

Closed
artie2000 wants to merge 165 commits intomasterfrom
real_closed_field_dev
Closed

[Merged by Bors] - feat(Algebra/Ring): generalise and extend material about sums of squares and semireal rings#16094
artie2000 wants to merge 165 commits intomasterfrom
real_closed_field_dev

Conversation

@artie2000
Copy link
Copy Markdown
Collaborator

@artie2000 artie2000 commented Aug 23, 2024

Deletions:

  • mem_sumSq_of_isSquare (statement changed to IsSquare.isSumSq)

This is work towards formalising some real algebra (see https://github.com/artie2000/real_closed_field).

Open in Gitpod

@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jan 13, 2025
@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jan 13, 2025
@YaelDillies
Copy link
Copy Markdown
Contributor

Thanks!

maintainer delegate

@github-actions
Copy link
Copy Markdown

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

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Jan 15, 2025
artie2000 and others added 5 commits January 15, 2025 22:46
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
Copy link
Copy Markdown
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Jan 16, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jan 16, 2025
…res and semireal rings (#16094)

* Generalise definition of a semireal ring, as discussed at https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/Formally.20real.20fields
* Add facts about multiplicative structure of sums of squares -- preserved under multiplication, form a semiring, set of sums of squares is the semiring generated by the squares.
* Add some lemmas to simpset
* Update documentation

Deletions:
* `mem_sumSq_of_isSquare` (statement changed to `IsSquare.isSumSq`)
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 16, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Algebra/Ring): generalise and extend material about sums of squares and semireal rings [Merged by Bors] - feat(Algebra/Ring): generalise and extend material about sums of squares and semireal rings Jan 16, 2025
@mathlib-bors mathlib-bors bot closed this Jan 16, 2025
@mathlib-bors mathlib-bors bot deleted the real_closed_field_dev branch January 16, 2025 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

large-import Automatically added label for PRs with a significant increase in transitive imports maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. ready-to-merge This PR has been sent to bors. t-algebra Algebra (groups, rings, fields, etc) t-order Order theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants