Skip to content

[Merged by Bors] - feat(Data/Sym/Sym2): Multiplication as a function from Sym2#21836

Closed
mans0954 wants to merge 11 commits intomasterfrom
mans0954/Finsupp-Sym2
Closed

[Merged by Bors] - feat(Data/Sym/Sym2): Multiplication as a function from Sym2#21836
mans0954 wants to merge 11 commits intomasterfrom
mans0954/Finsupp-Sym2

Conversation

@mans0954
Copy link
Copy Markdown
Collaborator

Used in #18578


Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 13, 2025

PR summary b0f72b98f7

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Data.Sym.Sym2 483 487 +4 (+0.83%)
Import changes for all files
Files Import difference
20 files Mathlib.Combinatorics.SimpleGraph.Acyclic Mathlib.Combinatorics.SimpleGraph.Connectivity.Represents Mathlib.Combinatorics.SimpleGraph.Connectivity.Subgraph Mathlib.Combinatorics.SimpleGraph.Connectivity.WalkCounting Mathlib.Combinatorics.SimpleGraph.DegreeSum Mathlib.Combinatorics.SimpleGraph.Density Mathlib.Combinatorics.SimpleGraph.Girth Mathlib.Combinatorics.SimpleGraph.Matching Mathlib.Combinatorics.SimpleGraph.Regularity.Energy Mathlib.Combinatorics.SimpleGraph.Regularity.Uniform Mathlib.Combinatorics.SimpleGraph.Triangle.Basic Mathlib.Combinatorics.SimpleGraph.Triangle.Tripartite Mathlib.Combinatorics.SimpleGraph.UniversalVerts Mathlib.SetTheory.Game.Birthday Mathlib.SetTheory.Game.Domineering Mathlib.SetTheory.Game.Nim Mathlib.SetTheory.Game.Ordinal Mathlib.SetTheory.Game.Short Mathlib.SetTheory.Game.State Mathlib.SetTheory.Surreal.Basic
1
6 files Mathlib.Combinatorics.SimpleGraph.Coloring Mathlib.Combinatorics.SimpleGraph.ConcreteColorings Mathlib.Combinatorics.SimpleGraph.Diam Mathlib.Combinatorics.SimpleGraph.Metric Mathlib.Combinatorics.SimpleGraph.Partition Mathlib.Combinatorics.SimpleGraph.Sum
3
29 files Mathlib.Algebra.BigOperators.Sym Mathlib.Combinatorics.Digraph.Orientation Mathlib.Combinatorics.SimpleGraph.Basic Mathlib.Combinatorics.SimpleGraph.Circulant Mathlib.Combinatorics.SimpleGraph.Clique Mathlib.Combinatorics.SimpleGraph.Dart Mathlib.Combinatorics.SimpleGraph.Finite Mathlib.Combinatorics.SimpleGraph.Hamiltonian Mathlib.Combinatorics.SimpleGraph.Hasse Mathlib.Combinatorics.SimpleGraph.LineGraph Mathlib.Combinatorics.SimpleGraph.Maps Mathlib.Combinatorics.SimpleGraph.Operations Mathlib.Combinatorics.SimpleGraph.Path Mathlib.Combinatorics.SimpleGraph.Prod Mathlib.Combinatorics.SimpleGraph.Subgraph Mathlib.Combinatorics.SimpleGraph.Trails Mathlib.Combinatorics.SimpleGraph.Turan Mathlib.Combinatorics.SimpleGraph.Walk Mathlib.Data.Finset.Sym Mathlib.Data.List.Sym Mathlib.Data.Multiset.Sym Mathlib.Data.Sym.Card Mathlib.Data.Sym.Sym2.Order Mathlib.Data.Sym.Sym2 Mathlib.Order.GameAdd Mathlib.SetTheory.Game.Basic Mathlib.SetTheory.Game.Impartial Mathlib.SetTheory.Game.PGame Mathlib.SetTheory.ZFC.Ordinal
4

Declarations diff

+ lift_smul_lift
+ mul
+ mul_mk

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).

@github-actions github-actions bot added the t-data Data (lists, quotients, numbers, etc) label Feb 13, 2025
@b-mehta b-mehta changed the title Feature(Data/Sym/Sym2): CommMagma.mul as a function from Sym2` feat(Data/Sym/Sym2): CommMagma.mul as a function from Sym2 Feb 13, 2025
@mans0954 mans0954 changed the title feat(Data/Sym/Sym2): CommMagma.mul as a function from Sym2 feat(Data/Sym/Sym2): CommMagma.toMul.mul as a function from Sym2 Feb 13, 2025
@mans0954 mans0954 changed the title feat(Data/Sym/Sym2): CommMagma.toMul.mul as a function from Sym2 feat(Data/Sym/Sym2): Multiplication as a function from Sym2 Feb 14, 2025
@YaelDillies YaelDillies added the awaiting-author A reviewer has asked the author a question or requested changes. label Feb 14, 2025
mans0954 and others added 2 commits February 14, 2025 17:46
@mans0954 mans0954 removed the awaiting-author A reviewer has asked the author a question or requested changes. label Feb 14, 2025
Copy link
Copy Markdown
Contributor

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

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

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 Feb 14, 2025
Copy link
Copy Markdown
Member

@TwoFX TwoFX left a comment

Choose a reason for hiding this comment

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

bors r+

@ghost ghost added ready-to-merge This PR has been sent to bors. and removed maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. labels Feb 18, 2025
mathlib-bors bot pushed a commit that referenced this pull request Feb 18, 2025
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 18, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Data/Sym/Sym2): Multiplication as a function from Sym2 [Merged by Bors] - feat(Data/Sym/Sym2): Multiplication as a function from Sym2 Feb 18, 2025
@mathlib-bors mathlib-bors bot closed this Feb 18, 2025
@mathlib-bors mathlib-bors bot deleted the mans0954/Finsupp-Sym2 branch February 18, 2025 20:32
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-data Data (lists, quotients, numbers, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants