Skip to content

[Merged by Bors] - feat: the category of ind-objects satisfies the AB5 axiom#21350

Closed
TwoFX wants to merge 1 commit intomasterfrom
fme-194
Closed

[Merged by Bors] - feat: the category of ind-objects satisfies the AB5 axiom#21350
TwoFX wants to merge 1 commit intomasterfrom
fme-194

Conversation

@TwoFX
Copy link
Copy Markdown
Member

@TwoFX TwoFX commented Feb 2, 2025


Open in Gitpod

@github-actions github-actions bot added the large-import Automatically added label for PRs with a significant increase in transitive imports label Feb 2, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 2, 2025

PR summary b30f21cad5

Import changes exceeding 2%

% File
+3.40% Mathlib.CategoryTheory.Limits.Preserves.Creates.Finite

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.CategoryTheory.Limits.Preserves.Creates.Finite 529 547 +18 (+3.40%)
Import changes for all files
Files Import difference
Mathlib.CategoryTheory.Generator.Indization Mathlib.CategoryTheory.Limits.Indization.Category 1
Mathlib.CategoryTheory.Limits.Preserves.Creates.Finite 18
Mathlib.CategoryTheory.Abelian.GrothendieckAxioms.Types (new file) 933
Mathlib.CategoryTheory.Abelian.GrothendieckAxioms.Indization (new file) 969

Declarations diff

+ hasFiniteColimits_of_hasColimits_of_createsFiniteColimits
+ hasFiniteLimits_of_hasLimitsLimits_of_createsFiniteLimits
+ instance (priority := 100) preservesFiniteColimits_of_createsFiniteColimits_and_hasFiniteColimits
+ instance (priority := 100) preservesFiniteLimits_of_createsFiniteLimits_and_hasFiniteLimits
+ instance : AB5 (Type v)
+ instance [HasFiniteColimits C] : HasColimits (Ind C)
+ instance [HasFiniteLimits C] : AB5 (Ind C)
+ instance [HasFiniteLimits C] : CreatesFiniteLimits (Ind.inclusion C)
+ instance [HasFiniteLimits C] : HasFiniteLimits (Ind C)
+ instance [HasLimits C] : CreatesLimitsOfSize.{v, v} (Ind.inclusion C)
+ instance [HasLimits C] : HasLimits (Ind C)
+ instance {J : Type v} [SmallCategory J] [IsFiltered J] [HasFiniteLimits C] :
++ instance [HasLimitsOfShape WalkingParallelPair C] :

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-category-theory Category theory label Feb 2, 2025
@dagurtomas
Copy link
Copy Markdown
Contributor

The import change is completely reasonable (side note: we should maybe move the file Limits.Creates into the folder Limits.Preserves.Creates at some point). Thanks!

maintainer merge

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 5, 2025

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

@github-actions github-actions bot added the maintainer-merge A reviewer has approved the changed; awaiting maintainer approval. label Feb 5, 2025
@joelriou
Copy link
Copy Markdown
Contributor

joelriou commented Feb 6, 2025

Thanks!

bors merge

@TwoFX
Copy link
Copy Markdown
Member Author

TwoFX commented Feb 6, 2025

Looks like the bot missed this one?

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 6, 2025
mathlib-bors bot pushed a commit that referenced this pull request Feb 6, 2025



Co-authored-by: Markus Himmel <markus@lean-fro.org>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 6, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat: the category of ind-objects satisfies the AB5 axiom [Merged by Bors] - feat: the category of ind-objects satisfies the AB5 axiom Feb 6, 2025
@mathlib-bors mathlib-bors bot closed this Feb 6, 2025
@mathlib-bors mathlib-bors bot deleted the fme-194 branch February 6, 2025 12:04
Julian added a commit that referenced this pull request Feb 7, 2025
* origin/master:
  chore: update Mathlib dependencies 2025-02-06 (#21523)
  fix(MathlibTest/TransImports): stop inspecting the `Lean` package (#21492)
  style(Mathlib/Computability/Halting): `RePred` to `REPred` (#21216)
  feat(Data/Set/Card): add `ncard_le_encard` (#21467)
  feat(Order): lemmas for `Order.succ` and `Order.pred` in `Fin` (#21437)
  feat(LinearAlgebra/LinearIndependent): linear independence + subsingletons (#21511)
  feat: for continuous linear maps in a normed ring, `flip mul = mul` (#21507)
  chore(GroupTheory/Commutator): don't import `Ring` (#21296)
  chore(Data/Complex/Abs): add `protected` to results that already exists in root namespace (#21454)
  chore(*): `erw`s that can now become `rw`s (#21510)
  chore: allow create-adaptation-pr.sh to continue when bump branch already exists (#21486)
  feat(CategoryTheory): equivalence between `Ind C` and left exact functors from `C` to `Type` (#21430)
  chore: add test to TCSynth.lean (#21499)
  feat: the category of ind-objects satisfies the AB5 axiom (#21350)
  refactor(RepresentationTheory): `ConcreteCategory` instances for `Rep` (#21465)
  chore: split Mathlib.Order.Filter.Basic (#21403)
  chore: update Mathlib dependencies 2025-02-06 (#21487)
  chore(Cache): Add support for $MATHLIB_CACHE_DIR (#21480)
  feat(CategoryTheory): a closed monoidal category is an ordinary enriched category over itself (#21436)
  feat(AlgebraicTopology): notation X ^[n] for cosimplicial objects (#21485)
  chore: upgrade dependencies manually (#21484)
  refactor(Analysis/Normed): `ConcreteCategory` refactor for `SemiNormedGrp` (#21477)
  refactor(LinearAlgebra): `ConcreteCategory` instance for `QuadraticModuleCat` (#21471)
  refactor(MeasureTheory): `ConcreteCategory` instance for `MeasCat` (#21468)
  refactor(Topology/Category): clean up remaining uses of `HasForget` (#21458)
  refactor(CategoryTheory): `ConcreteCategory` instances for pointed types (#21470)
  feat(CategoryTheory/Action): `ConcreteCategory` instances for `Action` (#21462)
  feat(CategoryTheory): `ConcreteCategory` instance for `DifferentialObject` (#21464)
  feat(Analysis/Normed/Group/SeparationQuotient): add normed lifts and `mk` (#18178)
  chore: rename `encard_le_card` to `encard_le_encard` (#21426)
  feat: add theorem about the norm of cross products (#20920)
  feat(Data/Matroid/Circuit): circuit elimination and finitary matroids (#21172)
  feat(LinearAlgebra/ExteriorPower): add iMulti_family definition for product of a family of vectors (#21397)
pfaffelh pushed a commit that referenced this pull request Feb 7, 2025



Co-authored-by: Markus Himmel <markus@lean-fro.org>
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 ready-to-merge This PR has been sent to bors. t-category-theory Category theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants