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

feat(Algebra/Ring): generalise and extend material about sums of squares and semireal rings #16094

Open
wants to merge 150 commits into
base: master
Choose a base branch
from

Conversation

artie2000
Copy link
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

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR to Mathlib (this label is automatically managed by a bot) label Jan 6, 2025
@artie2000 artie2000 removed the awaiting-author A reviewer has asked the author a question or requested changes label Jan 6, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jan 7, 2025
)

* Make documentation clearer and more concise
* Make notation consistent
* Make proofs cleaner

This PR splits off yet more chore material from #16094

Moves:
* `sumSq` -> `AddSubmonoid.sumSq`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked-by-other-PR This PR depends on another PR to Mathlib (this label is automatically managed by a bot) large-import Automatically added label for PRs with a significant increase in transitive imports 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