Skip to content

[Merged by Bors] - feat(CategoryTheory): the class of morphisms given by a family of maps#21354

Closed
joelriou wants to merge 2 commits intomasterfrom
morphism-property-ofhoms
Closed

[Merged by Bors] - feat(CategoryTheory): the class of morphisms given by a family of maps#21354
joelriou wants to merge 2 commits intomasterfrom
morphism-property-ofhoms

Conversation

@joelriou
Copy link
Copy Markdown
Contributor

@joelriou joelriou commented Feb 2, 2025 •
edited
Loading

Given a family of morphisms f i : X i ⟶ Y i in a category C, this PR introduces that the class of morphisms ofHoms f : MorphismProperty C consisting of these morphisms f i.

Open in Gitpod

@joelriou joelriou added the t-category-theory Category theory label Feb 2, 2025
@joelriou joelriou mentioned this pull request Feb 2, 2025
Closed
3 tasks
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 2, 2025 •
edited
Loading

PR summary a37b197329

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ Arrow.functor_ext
+ arrow_mk_comp_eqToHom
+ arrow_mk_eqToHom_comp
+ arrow_mk_mem_toSet_iff
+ homFamily
+ homFamily_apply
+ homFamily_arrow_mk
+ mk_eq_mk_iff
+ ofHoms
+ ofHoms_homFamily
+ ofHoms_iff
+ toSet

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

@joelriou joelriou changed the title feat(CategoryTheory): class of morphisms given by a family of maps feat(CategoryTheory): the class of morphisms given by a family of maps Feb 2, 2025
TwoFX
TwoFX approved these changes Feb 3, 2025
View reviewed changes
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 Feb 3, 2025
mathlib-bors bot pushed a commit that referenced this pull request Feb 3, 2025
@joelriou
feat(CategoryTheory): the class of morphisms given by a family of maps (
b76b14c 
#21354)

Given a family of morphisms `f i : X i ⟶ Y i` in a category `C`, this PR introduces that the class of morphisms `ofHoms f : MorphismProperty C` consisting of these morphisms `f i`.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Feb 3, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(CategoryTheory): the class of morphisms given by a family of maps [Merged by Bors] - feat(CategoryTheory): the class of morphisms given by a family of maps Feb 3, 2025
@mathlib-bors mathlib-bors bot closed this Feb 3, 2025
@mathlib-bors mathlib-bors bot deleted the morphism-property-ofhoms branch February 3, 2025 05:49
jt496 pushed a commit that referenced this pull request Feb 3, 2025
@joelriou @jt496
feat(CategoryTheory): the class of morphisms given by a family of maps (
288ca9c 
#21354)

Given a family of morphisms `f i : X i ⟶ Y i` in a category `C`, this PR introduces that the class of morphisms `ofHoms f : MorphismProperty C` consisting of these morphisms `f i`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TwoFX TwoFX TwoFX approved these changes

Assignees

No one assigned

Labels

ready-to-merge This PR has been sent to bors. t-category-theory Category theory

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@joelriou @TwoFX