Skip to content

[Merged by Bors] - feat(CategoryTheory): condition for an induced functor between comma categories to be final #20139

Closed
javra wants to merge 15 commits intomasterfrom
fme-95
Closed

[Merged by Bors] - feat(CategoryTheory): condition for an induced functor between comma categories to be final #20139
javra wants to merge 15 commits intomasterfrom
fme-95

Conversation

@javra
Copy link
Copy Markdown
Collaborator

@javra javra commented Dec 20, 2024


Open in Gitpod

@javra javra added the WIP Work in progress label Dec 20, 2024
@github-actions github-actions bot added the t-category-theory Category theory label Dec 20, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 20, 2024

PR summary 0888d1c7eb

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.CategoryTheory.Comma.Final 584 588 +4 (+0.68%)
Import changes for all files
Files Import difference
Mathlib.CategoryTheory.Comma.Final 4
Mathlib.CategoryTheory.Comma.StructuredArrow.CommaMap (new file) 382

Declarations diff

+ StructuredArrow.final_map
+ commaMapEquivalence
+ commaMapEquivalenceCounitIso
+ commaMapEquivalenceFunctor
+ commaMapEquivalenceInverse
+ commaMapEquivalenceUnitIso
+ mapIsoMap₂
+ map_final
+ map₂CompMap₂Iso
+ preIsoMap₂

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

@javra javra removed the WIP Work in progress label Dec 22, 2024
@javra javra requested review from TwoFX and joelriou December 22, 2024 18:31
@javra
Copy link
Copy Markdown
Collaborator Author

javra commented Dec 23, 2024

Two questions:

  • Is the performance of StructuredArrow.commaMapEquivalence okay? Not sure how to make it much faster.
  • map₂CompMap₂Iso references the pasting of two square-shaped two cells, which @emilyriehl defined for adjoints, but I think it should be an operation independent of adjoints, maybe in CategoryTheory.whiskering? Might be worth doing that in a future refactor though.

@joelriou
Copy link
Copy Markdown
Contributor

* Is the performance of `StructuredArrow.commaMapEquivalence` okay? Not sure how to make it much faster.

It seems ok to me.

* `map₂CompMap₂Iso` references the pasting of two square-shaped two cells, which @emilyriehl defined for adjoints, but I think it should be an operation independent of adjoints, maybe in `CategoryTheory.whiskering`? Might be worth doing that in a future refactor though.

We have CatCommSq in the file CategoryTheory.CatCommSq.

@joelriou joelriou added the awaiting-author A reviewer has asked the author a question or requested changes. label Dec 23, 2024
@javra
Copy link
Copy Markdown
Collaborator Author

javra commented Dec 24, 2024

@joelriou Thanks! Didn't find that. We don't require the two cells to be iso, though.

@javra javra removed the awaiting-author A reviewer has asked the author a question or requested changes. label Dec 24, 2024
@javra javra requested a review from joelriou December 24, 2024 11:41
@joelriou
Copy link
Copy Markdown
Contributor

@joelriou Thanks! Didn't find that. We don't require the two cells to be iso, though.

Then, we have TwoSquare which appears in CategoryTheory.GuitartExact.Basic/VerticalComposition. It could be worth moving out the (very little) content that is specific to 2-squares from this file and develop a little more the API.

@javra
Copy link
Copy Markdown
Collaborator Author

javra commented Dec 28, 2024

Will do so in a separate PR (will also make sure naming etc is consistent with the general bicategorical case...)

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.

Thanks!
bors r+

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Jan 18, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jan 18, 2025
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 18, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(CategoryTheory): condition for an induced functor between comma categories to be final [Merged by Bors] - feat(CategoryTheory): condition for an induced functor between comma categories to be final Jan 18, 2025
@mathlib-bors mathlib-bors bot closed this Jan 18, 2025
@mathlib-bors mathlib-bors bot deleted the fme-95 branch January 18, 2025 19:36
Julian added a commit that referenced this pull request Jan 20, 2025
* polynomial-sequences: (149 commits)
  Aha, here's how to get Lean to stop showing S.elems' in the infoview.
  Try satisfying the linter gods again.
  Probably enough initial tidying to send the PR.
  Kill more temporary names.
  Touch more natDegree.
  Does protected satisfy the docstring linter?
  Bit shorter.
  More
  Quiet linters.
  Remove redundant imports.
  Copyright header and more twiddling.
  Rename lemma to 'degree_smul_of_leadingCoeff_rightRegular' and split out
  feat(Polynomial): polynomial sequences are bases for R[X]
  chore(Dynamics/PeriodicPts): don't import `MonoidWithZero` (#20765)
  chore(Associated): split out `Ring` results (#20737)
  feat(AlgebraicGeometry): flat morphisms of schemes (#19790)
  feat(AlgebraicGeometry): scheme-theoretic fibre (#19427)
  chore: split Mathlib.Analysis.Asymptotics.Asymptotics (#20785)
  doc: typo (#20829)
  feat(CategoryTheory): condition for an induced functor between comma categories to be final  (#20139)
  feat(1000.yaml): allow statements of theorems also (#20637)
  feat(Algebra/Homology/Embedding): homology of truncGE' (#19570)
  chore: cleanup many porting notes in Combinatorics (#20823)
  chore: eliminate porting notes about `deriving Fintype` (#20820)
  feat(Algebra/Lie): a Lie algebra is solvable iff it is solvable after faithfully flat base change (#20808)
  feat: define bases of root pairings (#20667)
  feat(Tactic): basic ConcreteCategory support for elementwise (#20811)
  feat(CategoryTheory): define unbundled `ConcreteCategory` class  (#20810)
  chore(CategoryTheory): rename `ConcreteCategory` to `HasForget` (#20809)
  feat: `CommSemiring (NonemptyInterval ℚ≥0)` (#20783)
  chore(yaml_check.py): re-format (#20807)
  feat: elementary estimate for Real.log (#20766)
  feat: definition of linear topologies (#14990)
  feat(RingTheory): flatness over a semiring (#19115)
  feat(Algebra/Homology/Embedding): the canonical truncation truncLE (#19550)
  feat(Algebra/Homology/Embedding): API for the homology of an extension of homological complex (#19203)
  feat(Algebra/Ring): `RingEquiv.piUnique` (#20794)
  feat(RingTheory/LocalRing): add instance `Unique (MaximalSpectrum R)` for a local ring `R` (#20801)
  chore(GroupExtension/Defs): define `Section` and redefine `Splitting` (#20802)
  chore: restore `def` to `adicCompletion` (#20796)
  refactor: rename `UniqueContinuousFunctionalCalculus` to `ContinuousMap.UniqueHom` (#20643)
  feat(Algebra/Homology/Embedding): the morphism from a complex to its `truncGE` truncation (#19544)
  chore(Mathlib/Computability/TuringMachine): split file (#20790)
  feat(Data/Finset/Card): add `InjOn` and `SurjOn` versions of finset cardinality lemmas (#20753)
  feat(Order/WellFoundedSet): add convenience constructors for IsWF and IsPWO for WellFoundedLT types (#20752)
  feat(Set/Finite): a set is finite if its image and fibers are finite (#20751)
  chore: cleanup .gitignore files (#20795)
  feat(Topology/Group/Profinite):  Profinite group is limit of finite group (#16992)
  feat(Combinatorics/SimpleGraph): vertices in cycles (#20602)
  CI: merge `bot_fix_style` actions (#20789)
  ...
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-category-theory Category theory

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants