Skip to content

[Merged by Bors] - feat(Topology/Algebra/InfiniteSum): use NormMulClass#28756

Closed
loefflerd wants to merge 2 commits intoleanprover-community:masterfrom
loefflerd:DL_norm_tprod
Closed

[Merged by Bors] - feat(Topology/Algebra/InfiniteSum): use NormMulClass#28756
loefflerd wants to merge 2 commits intoleanprover-community:masterfrom
loefflerd:DL_norm_tprod

Conversation

@loefflerd
Copy link
Copy Markdown
Contributor

@loefflerd loefflerd commented Aug 21, 2025

Prove norm_tprod and its cousins for any normed ring with a strictly multiplicative norm (not just fields).


Spun off from review process of #28400.
Open in Gitpod

@github-actions github-actions bot added the t-topology Topological spaces, uniform spaces, metric spaces, filters label Aug 21, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 21, 2025

PR summary 1f2e1c9e59

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Topology.Algebra.InfiniteSum.Field 1292 1291 -1 (-0.08%)
Import changes for all files
Files Import difference
Mathlib.Topology.Algebra.InfiniteSum.Field -1

Declarations diff

No declarations were harmed in the making of this PR! 🐙

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


No changes to technical debt.

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@CBirkbeck
Copy link
Copy Markdown
Collaborator

LGTM!

Copy link
Copy Markdown
Contributor

@b-mehta b-mehta 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 Aug 22, 2025
mathlib-bors bot pushed a commit that referenced this pull request Aug 22, 2025
Prove `norm_tprod` and its cousins for any normed ring with a strictly multiplicative norm (not just fields).
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 22, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Topology/Algebra/InfiniteSum): use NormMulClass [Merged by Bors] - feat(Topology/Algebra/InfiniteSum): use NormMulClass Aug 22, 2025
@mathlib-bors mathlib-bors bot closed this Aug 22, 2025
Paul-Lez pushed a commit to Paul-Lez/mathlib4 that referenced this pull request Aug 23, 2025
…unity#28756)

Prove `norm_tprod` and its cousins for any normed ring with a strictly multiplicative norm (not just fields).
pechersky pushed a commit to pechersky/mathlib4 that referenced this pull request Aug 25, 2025
…unity#28756)

Prove `norm_tprod` and its cousins for any normed ring with a strictly multiplicative norm (not just fields).
@loefflerd loefflerd deleted the DL_norm_tprod branch September 3, 2025 09:55
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-topology Topological spaces, uniform spaces, metric spaces, filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants