Skip to content

[Merged by Bors] - feat(Topology/Category): category of delta-generated spaces#19499

Closed
peabrainiac wants to merge 16 commits intomasterfrom
DeltaGenerated
Closed

[Merged by Bors] - feat(Topology/Category): category of delta-generated spaces#19499
peabrainiac wants to merge 16 commits intomasterfrom
DeltaGenerated

Conversation

@peabrainiac
Copy link
Copy Markdown
Collaborator

@peabrainiac peabrainiac commented Nov 26, 2024

Introduces the category of delta-generated spaces and shows that it is complete, cocomplete and coreflective in the category of topological spaces.


Open in Gitpod

@peabrainiac peabrainiac added t-category-theory Category theory t-topology Topological spaces, uniform spaces, metric spaces, filters labels Nov 26, 2024
@github-actions github-actions bot added the new-contributor This PR was made by a contributor with at most 5 merged PRs. Welcome to the community! label Nov 26, 2024
@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 26, 2024

PR summary f15b416599

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference
Mathlib.Topology.Category.DeltaGenerated (new file) 1484

Declarations diff

+ DeltaGenerated
+ coreflectorAdjunction
+ deltaGeneratedToTop
+ deltaGeneratedToTop.coreflective
+ deltaGeneratedToTop.createsColimits
+ fullyFaithfulDeltaGeneratedToTop
+ hasColimits
+ hasLimits
+ instance : CoeSort DeltaGenerated Type*
+ instance : ConcreteCategory.{u} DeltaGenerated.{u}
+ instance : LargeCategory.{u} DeltaGenerated.{u}
+ instance : deltaGeneratedToTop.{u}.Faithful := fullyFaithfulDeltaGeneratedToTop.faithful
+ instance : deltaGeneratedToTop.{u}.Full := fullyFaithfulDeltaGeneratedToTop.full
+ instance : topToDeltaGenerated.{u}.Faithful
+ of
+ topToDeltaGenerated

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

@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot added the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Nov 26, 2024
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Nov 26, 2024
@mathlib4-dependent-issues-bot
Copy link
Copy Markdown
Collaborator

This PR/issue depends on:

@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 9, 2024
@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Dec 11, 2024

Checking in from PR triage: this PR has a merge conflict. Can you merge master, please? Then this PR can be reviewed much more easily. Thanks!

@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Dec 11, 2024
@peabrainiac
Copy link
Copy Markdown
Collaborator Author

The merge conflict is fixed now. Thanks for pointing it out, I didn't notice - really unfortunate that github apparently doesn't give PR authors the option to get notified when merge confllicts occur.

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Dec 12, 2024

Thanks. I agree this is unfortunate. I can only think of two strategies:

  • in this specific case, a dependent PR was merged: that's usually an occasion for merge conflicts. Github's email notifications cover this.
  • for the general case, the on the queue page contains data about all open PRs: filtering by your github handle will give you an overview about all your PRs. (This might be the most useful if you have many PRs. I use this sometimes.)

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Dec 12, 2024

I just took a look at your PR. I just pushed a few nits myself (to save you the extra round of suggesting them and you accepting them); feel free to revert any changes you disagree with.

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Dec 12, 2024

Two follow-up questions:

  • how difficult is it to show that delta-generated spaces are compactly generated? DeltaGeneratedSpace mentions this, but doesn't prove it (as far as I can see)
  • given this PR, how far is mathlib from knowing that a diffeological space/an orbifold is? :-)

@grunweg
Copy link
Copy Markdown
Contributor

grunweg commented Dec 12, 2024

@kim-em This PR looks good to me: the proofs are reasonable, its structure matches CompactlyGenerated.lean. I'm not an expert on the best spelling of things in category theory. Can you give this a final look, please? Thanks!

@grunweg grunweg requested a review from kim-em December 12, 2024 12:27
@peabrainiac
Copy link
Copy Markdown
Collaborator Author

peabrainiac commented Dec 12, 2024

Two follow-up questions:

  • how difficult is it to show that delta-generated spaces are compactly generated? DeltaGeneratedSpace mentions this, but doesn't prove it (as far as I can see)

  • given this PR, how far is mathlib from knowing that a diffeological space/an orbifold is? :-)

Showing that they are compactly generated requires showing that they are generated not just by euclidean spaces like how I've defined them there, but also by something like the unit interval or the standard simplices. That is unfortunately tricky, because the only proof I know requires a concatenation of countably many paths - I've already written the proof up, but the details of this concatenation are unfortunately hairy, so in it I have a few sorries left.

Diffeological spaces on the other hand have honestly been in reach the entire time - I just somewhat procrastinated them, both because PRing their definition feels a lot like locking in design choices that I am not yet 100% sure about, and because there are some things about them that I not not super happy with. For example, that the d-topology and quotient topology on a diffeological quotient space agree is currently only a theorem and not true by rfl, which is quite inconvenient - but changing that would require defining coinduced diffeologies only after the subspace diffeology has already been developed, which would require some restructuring across files.

@peabrainiac
Copy link
Copy Markdown
Collaborator Author

peabrainiac commented Dec 12, 2024

Showing that they are compactly generated requires showing that they are generated not just by euclidean spaces like how I've defined them there, but also by something like the unit interval or the standard simplices.

Actually thinking about it, this of course isn't true: every delta-generated space is a quotient of a disjoint union of compactly generated spaces, and so itself compactly generated. I do need the path stuff anyway to show that first-countable locally path-connected spaces are delta-generated, but at least here it isn't necessary.

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Jan 11, 2025

bors merge

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Jan 11, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jan 11, 2025
Introduces the category of delta-generated spaces and shows that it is complete, cocomplete and coreflective in the category of topological spaces.



Co-authored-by: peabrainiac <43812953+peabrainiac@users.noreply.github.com>
Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 11, 2025

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Jan 11, 2025
Introduces the category of delta-generated spaces and shows that it is complete, cocomplete and coreflective in the category of topological spaces.



Co-authored-by: peabrainiac <43812953+peabrainiac@users.noreply.github.com>
Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 11, 2025

Build failed (retrying...):

mathlib-bors bot pushed a commit that referenced this pull request Jan 11, 2025
Introduces the category of delta-generated spaces and shows that it is complete, cocomplete and coreflective in the category of topological spaces.



Co-authored-by: peabrainiac <43812953+peabrainiac@users.noreply.github.com>
Co-authored-by: Michael Rothgang <rothgang@math.uni-bonn.de>
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jan 11, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(Topology/Category): category of delta-generated spaces [Merged by Bors] - feat(Topology/Category): category of delta-generated spaces Jan 11, 2025
@mathlib-bors mathlib-bors bot closed this Jan 11, 2025
@mathlib-bors mathlib-bors bot deleted the DeltaGenerated branch January 11, 2025 04:29
Julian added a commit that referenced this pull request Jan 12, 2025
* origin/master: (88 commits)
  chore(scripts): update nolints.json (#20672)
  chore: de-simp `map_eq_zero_iff_eq_one` (#20662)
  feat(Combinatorics/SimpleGraph): add independent sets (#18608)
  chore(CategoryTheory/Limits/Cones): functoriality of `mapCone` (#20641)
  feat(Algebra/Category/ModuleCat): pullback of presheaves of modules (#17366)
  feat(AlgebraicTopology): model categories (#19158)
  chore(CategoryTheory): make NormalEpi/MonoCategory and RegularEpi/MonoCategory props (#19548)
  feat(Data/List/ReduceOption): add replicate theorems (#20644)
  feat: approximate subgroups (#20050)
  feat: use scoped trace nodes in linarith (#19855)
  feat: disjoint union of charted spaces (#20619)
  feat: add some term elaborators for reduction (#15192)
  feat(Topology/Category): category of delta-generated spaces (#19499)
  add a variable_alias for Quantale and AddQuantale (#19282)
  feat(Computability/DFA): implement `isRegular_iff` (#19940)
  chore: unpin and bump batteries and importgraph (#20651)
  chore: split `Mathlib/Algebra/Group/Int` (#20624)
  feat: three lemmas related to Hausdorff distance (#20585)
  chore: `initialize_simps_projections` for `Submodule` (#20582)
  feat(Order): Boolean algebra structure on idempotents (#20618)
  chore(CategoryTheory): moving/renaming Subpresheaf (#20583)
  refactor(IntermediateField/Adjoin): Split off relation to `Algebra.adjoin` (#20630)
  feat: sets of doubling strictly less than 3/2 (#20572)
  chore(TensorProduct): universe polymorphism in EquationalCriterion (#20452)
  feat: `s \ t ∩ u = (s ∩ u) \ t` (#20298)
  feat: product of subalgebras (#20202)
  feat: `Submodule.restrictScalars` commutes with `pow` (#20581)
  feat: `a ∈ s ^ n` iff there exists a sequence `f` of `n` elements of `s` such that `∏ i, f i = a` (#20580)
  chore: make `FooHom.coe_id` a `norm_cast` lemma (#20576)
  chore: use ofNat more (#20546)
  feat(CategoryTheory/Shift/Opposite and CategoryTheory/Shift/Pullback): `CommShift` structures on adjunctions are compatible with opposites and pullbacks (#20363)
  feat(FieldTheory/Differential/Liouville): prove the algebraic case of Liouville's theorem (#16797)
  refactor: remove the `CompactSpace` field from `Unique{NonUnital}ContinuousFunctionalCalculus` (#20590)
  feat: Make `PNat.recOn` induction eliminator (#20617)
  feat(Analysis/SpecialFunctions/Pow/Real): add some lemmas (#20608)
  feat: If `s ∆ t` is finite, then `s ∆ u` is finite iff `t ∆ u` is (#20574)
  feat: `⨅ i, f i ≤ ⨆ i, f i` (#20573)
  chore(Geometry/Manifold): move SmoothManifoldWithCorners.lean to IsManifold.lean (#20611)
  feat: AbsoluteValue.IsNontrivial (#20588)
  chore(Data/Finsupp): split off extensionality from `Defs.lean` (#19092)
  chore(Data/Set): split the `CoeSort` instance to its own file (#19031)
  feat(Algebra/Order/Archimedean/Basic): powers between two elements (#20612)
  feature(Algebra/Ring/Idempotents): product of an idempotent and its complement (#20286)
  chore: cleanup more `erw` (#20601)
  chore(GroupTheory/CoprodI): shorten proof of lift_word_prod_nontrivial_of_not_empty (#20587)
  chore: cleanup imports in PrimePow/Divisors (#20626)
  chore: split Algebra/BigOperators/Group/List (#20625)
  chore: reduce Topology->Order imports by moving content (#20627)
  chore(Algebra/Lie/DirectSum): shorten proof of lieAlgebraOf.map_lie' (#20592)
  refactor: Split `FieldTheory/Adjoin.lean` into `Defs.lean` and `Basic.lean` (#20333)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-contributor This PR was made by a contributor with at most 5 merged PRs. Welcome to the community! ready-to-merge This PR has been sent to bors. t-category-theory Category theory t-topology Topological spaces, uniform spaces, metric spaces, filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants