Skip to content

[Merged by Bors] - chore (Algebra.Order.Group.Int): split file into unbundled and bundled ordered algebra#15069

Closed
mattrobball wants to merge 9 commits intomasterfrom
mrb/split_algebra_order_group_int
Closed

[Merged by Bors] - chore (Algebra.Order.Group.Int): split file into unbundled and bundled ordered algebra#15069
mattrobball wants to merge 9 commits intomasterfrom
mrb/split_algebra_order_group_int

Conversation

@mattrobball
Copy link
Copy Markdown
Contributor

@mattrobball mattrobball commented Jul 23, 2024

All but ~5 lines of this file did not require bundled ordered algebra classes. We split them off into Algebra.Order.Group.Unbundled.Int leaving the LinearOrderedAddCommGroup instance.


Open in Gitpod

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 23, 2024

PR summary 98c722197d

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.Algebra.Order.Group.Int 347 325 -22 (-6.34%)
Mathlib.Algebra.EuclideanDomain.Int 379 373 -6 (-1.58%)
Mathlib.Algebra.Order.Ring.Int 392 388 -4 (-1.02%)
Mathlib.Data.Int.Interval 543 547 +4 (+0.74%)
Import changes for all files
Files Import difference
Mathlib.Algebra.Order.Group.Int -22
Mathlib.Algebra.EuclideanDomain.Int -6
8 files Mathlib.Data.Int.ConditionallyCompleteOrder Mathlib.Data.Int.LeastGreatest Mathlib.Algebra.Order.Ring.Int Mathlib.Testing.SlimCheck.Testable Mathlib.Testing.SlimCheck.Sampleable Mathlib.Data.Int.Range Mathlib.Algebra.Order.Ring.Rat Mathlib.Tactic.SlimCheck
-4
3 files Mathlib.Order.Grade Mathlib.Data.Nat.Cast.SetInterval Mathlib.Data.Int.SuccPred
-2
106 files Mathlib.Tactic.Ring.Basic Mathlib.Data.Nat.Hyperoperation Mathlib.Algebra.Star.Module Mathlib.Data.Matrix.Auto Mathlib.Data.Finset.Interval Mathlib.LinearAlgebra.Finsupp Mathlib.Data.Finset.Grade Mathlib.Data.Finsupp.Basic Mathlib.Tactic.Ring.PNat Mathlib.Data.Matrix.Invertible Mathlib.Tactic.Linarith.Preprocessing Mathlib.CategoryTheory.Monoidal.Linear Mathlib.Data.Matrix.Composition Mathlib.Tactic.LinearCombination Mathlib.Data.Matrix.Basic Mathlib.Data.Matrix.Basis Mathlib.GroupTheory.Coxeter.Matrix Mathlib.LinearAlgebra.DFinsupp Mathlib.Data.Finsupp.Encodable Mathlib.Data.Matrix.PEquiv Mathlib.Algebra.Category.ModuleCat.Monoidal.Symmetric Mathlib.LinearAlgebra.Dimension.Finrank Mathlib.Algebra.Category.ModuleCat.Monoidal.Basic Mathlib.Algebra.MonoidAlgebra.Support Mathlib.Data.Rat.Denumerable Mathlib.RingTheory.Coprime.Lemmas Mathlib.Data.Nat.Fib.Basic Mathlib.Tactic.Polyrith Mathlib.Tactic.Linarith.Frontend Mathlib.Data.Rat.BigOperators Mathlib.Algebra.Algebra.Hom.Rat Mathlib.Data.Matrix.Block Mathlib.Algebra.Order.Ring.Star Mathlib.Algebra.Field.Rat Mathlib.Algebra.Algebra.Unitization Mathlib.Data.Rat.Cast.Defs Mathlib.CategoryTheory.Preadditive.Mat Mathlib.CategoryTheory.Quotient.Linear Mathlib.Algebra.MonoidAlgebra.Degree Mathlib.Combinatorics.SetFamily.Compression.UV Mathlib.Data.Matrix.Reflection Mathlib.Algebra.Ring.Identities Mathlib.Testing.SlimCheck.Functions Mathlib.LinearAlgebra.Dimension.Basic Mathlib.LinearAlgebra.LinearIndependent Mathlib.Tactic.Ring.RingNF Mathlib.Tactic.Linarith.Oracle.FourierMotzkin Mathlib.Algebra.Star.Rat Mathlib.Data.Nat.BitIndices Mathlib.Algebra.Ring.BooleanRing Mathlib.Algebra.Homology.ShortComplex.Linear Mathlib.Data.NNRat.Lemmas Mathlib.Combinatorics.SetFamily.Shadow Mathlib.LinearAlgebra.Matrix.Trace Mathlib.Algebra.Star.Order Mathlib.RepresentationTheory.Action.Monoidal Mathlib.RepresentationTheory.Action.Limits Mathlib.Data.PNat.Xgcd Mathlib.Tactic.NormNum.Inv Mathlib.Data.FP.Basic Mathlib.Algebra.Category.ModuleCat.Monoidal.Closed Mathlib.Tactic.Group Mathlib.Tactic.NormNum.Eq Mathlib.Data.Rat.Cast.Lemmas Mathlib.CategoryTheory.Monoidal.Subcategory Mathlib.Algebra.Star.SelfAdjoint Mathlib.Tactic.Linarith.Parsing Mathlib.Tactic.NormNum.OfScientific Mathlib.Algebra.Order.Field.Rat Mathlib.LinearAlgebra.Matrix.Symmetric Mathlib.Tactic.Linarith.Datatypes Mathlib.Order.Interval.Finset.Box Mathlib.Order.KonigLemma Mathlib.Data.NNRat.Defs Mathlib.Tactic.Linarith.Oracle.SimplexAlgorithm Mathlib.Algebra.ContinuedFractions.Determinant Mathlib.Data.Rat.Cast.CharZero Mathlib.Algebra.Homology.Linear Mathlib.Algebra.MonoidAlgebra.Division Mathlib.Data.Fintype.Perm Mathlib.Tactic.Ring Mathlib.RingTheory.Coprime.Basic Mathlib.Algebra.Polynomial.Monomial Mathlib.Data.Matrix.RowCol Mathlib.Data.Matrix.DualNumber Mathlib.CategoryTheory.Linear.LinearFunctor Mathlib.Data.Finsupp.ToDFinsupp Mathlib.Algebra.Field.Subfield Mathlib.Data.Matrix.Hadamard Mathlib.Data.Nat.Fib.Zeckendorf Mathlib.Algebra.Polynomial.Basic Mathlib.Algebra.Algebra.Field Mathlib.Algebra.Algebra.Rat Mathlib.Algebra.FreeNonUnitalNonAssocAlgebra Mathlib.Algebra.MonoidAlgebra.Basic Mathlib.Data.Finsupp.AList Mathlib.Combinatorics.SimpleGraph.IncMatrix Mathlib.Tactic.Linarith.Verification Mathlib.Algebra.Category.BoolRing Mathlib.Algebra.Module.LinearMap.Rat Mathlib.Algebra.Order.Field.Subfield Mathlib.Data.Matrix.Notation Mathlib.Algebra.Module.Rat Mathlib.Tactic.Bound Mathlib.LinearAlgebra.Matrix.Circulant Mathlib.LinearAlgebra.Matrix.Orthogonal
-1
Mathlib.Data.Int.Interval 4
Mathlib.Algebra.Order.Group.Unbundled.Int 341

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.

@ghost ghost added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jul 23, 2024
@ghost ghost removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Jul 24, 2024
@ghost
Copy link
Copy Markdown

ghost commented Jul 24, 2024

@ghost ghost added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jul 29, 2024
@ghost ghost removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Jul 29, 2024
@j-loreaux
Copy link
Copy Markdown
Contributor

I'm delegating this in case we want to wait until after the new variable inclusion mechanism, as I imagine splits might make PR harder. But maybe not, I'll let you decide.

bors d+

@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 8, 2024

✌️ mattrobball can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@github-actions github-actions bot added the delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). label Aug 8, 2024
@mattrobball
Copy link
Copy Markdown
Contributor Author

Since this is about a concrete type, Int, there are not many parameters to the statements.

@mattrobball
Copy link
Copy Markdown
Contributor Author

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Aug 9, 2024
mathlib-bors bot pushed a commit that referenced this pull request Aug 9, 2024
…d ordered algebra (#15069)

All but ~5 lines of this file did not require bundled ordered algebra classes. We split them off into `Algebra.Order.Group.Unbundled.Int` leaving the `LinearOrderedAddCommGroup` instance.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Aug 9, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore (Algebra.Order.Group.Int): split file into unbundled and bundled ordered algebra [Merged by Bors] - chore (Algebra.Order.Group.Int): split file into unbundled and bundled ordered algebra Aug 9, 2024
@mathlib-bors mathlib-bors bot closed this Aug 9, 2024
@mathlib-bors mathlib-bors bot deleted the mrb/split_algebra_order_group_int branch August 9, 2024 12:27
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
…d ordered algebra (#15069)

All but ~5 lines of this file did not require bundled ordered algebra classes. We split them off into `Algebra.Order.Group.Unbundled.Int` leaving the `LinearOrderedAddCommGroup` instance.
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 9, 2024
…d ordered algebra (#15069)

All but ~5 lines of this file did not require bundled ordered algebra classes. We split them off into `Algebra.Order.Group.Unbundled.Int` leaving the `LinearOrderedAddCommGroup` instance.
bjoernkjoshanssen pushed a commit that referenced this pull request Sep 12, 2024
…d ordered algebra (#15069)

All but ~5 lines of this file did not require bundled ordered algebra classes. We split them off into `Algebra.Order.Group.Unbundled.Int` leaving the `LinearOrderedAddCommGroup` instance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

delegated This pull request has been delegated to the PR author (or occasionally another non-maintainer). ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants