Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(CategoryTheory/Sites): make Subcanonical a class #18186

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

dagurtomas
Copy link
Collaborator


Open in Gitpod

@dagurtomas dagurtomas added the WIP Work in progress label Oct 24, 2024
@dagurtomas dagurtomas changed the title chore(CategoryTheory/Sites): make Subcanonical a class chore(CategoryTheory/Sites): make Subcanonical a class Oct 24, 2024
Copy link

github-actions bot commented Oct 24, 2024

PR summary d039e26c85

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

+ Sheaf.Subcanonical
+ Sheaf.Subcanonical.isSheaf_of_isRepresentable
+ Sheaf.Subcanonical.of_isSheaf_yoneda_obj
+ Sheaf.Subcanonical.yoneda
+ Sheaf.Subcanonical.yonedaCompSheafToPresheaf
+ Sheaf.Subcanonical.yonedaFullyFaithful
+ instance : (J.yoneda).Faithful := (J.yonedaFullyFaithful).faithful
+ instance : (J.yoneda).Full := (J.yonedaFullyFaithful).full
+ instance : (canonicalTopology C).Subcanonical
+ instance : compHausToCondensed.Faithful := inferInstanceAs (_ ⋙ _).Faithful
+ instance : compHausToCondensed.Full := inferInstanceAs (_ ⋙ _).Full
+ le_canonical
+ of_isSheaf_yoneda_obj
++-- subcanonical
- instance : compHausToCondensed.Faithful := show (_ ⋙ _).Faithful from inferInstance
- instance : compHausToCondensed.Full := show (_ ⋙ _).Full from inferInstance
- instance : hJ.yoneda.Faithful := hJ.yonedaFullyFaithful.faithful
- instance : hJ.yoneda.Full := hJ.yonedaFullyFaithful.full
- of_yoneda_isSheaf

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.

@dagurtomas
Copy link
Collaborator Author

!bench

@dagurtomas dagurtomas removed the WIP Work in progress label Oct 29, 2024
@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit 4693fc8.
There were significant changes against commit f7b57bc:

  Benchmark      Metric                 Change
  =====================================================
- build          .olean serialization    11.2%
- build          import                  11.6%
- build          initialization           9.6%
- open Mathlib   task-clock              18.1% (78.4 σ)

Subcanonical.le_canonical

instance : (canonicalTopology C).Subcanonical where
le_canonical := le_rfl

namespace Subcanonical

/-- If every functor `yoneda.obj X` is a `J`-sheaf, then `J` is subcanonical. -/
theorem of_yoneda_isSheaf (J : GrothendieckTopology C)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This lemma could be renamed yoneda_obj_isSheaf.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I renamed it to of_isSheaf_yoneda_obj, which seems to be the most correct

@joelriou
Copy link
Collaborator

Here are the benchmark results for commit 4693fc8. There were significant changes against commit f7b57bc:

  Benchmark      Metric                 Change
  =====================================================
- build          .olean serialization    11.2%
- build          import                  11.6%
- build          initialization           9.6%
- open Mathlib   task-clock              18.1% (78.4 σ)

I am completely confused about what this means!

@joelriou joelriou added the awaiting-author A reviewer has asked the author a question or requested changes label Oct 30, 2024
@dagurtomas
Copy link
Collaborator Author

Here are the benchmark results for commit 4693fc8. There were significant changes against commit f7b57bc:

  Benchmark      Metric                 Change
  =====================================================
- build          .olean serialization    11.2%
- build          import                  11.6%
- build          initialization           9.6%
- open Mathlib   task-clock              18.1% (78.4 σ)

I am completely confused about what this means!

So am I... I'll try again

@dagurtomas
Copy link
Collaborator Author

!bench

@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit ac94109.
There were significant changes against commit f7b57bc:

  Benchmark      Metric                 Change
  =====================================================
- build          .olean serialization    11.2%
- build          import                  11.3%
- build          initialization           9.3%
- open Mathlib   task-clock              16.3% (11.5 σ)
- open Mathlib   wall-clock              14.6% (10.6 σ)

@dagurtomas
Copy link
Collaborator Author

!bench

@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit d039e26.
There were no significant changes against commit 7c7a48f.

@dagurtomas
Copy link
Collaborator Author

Ok, merging master got rid of these weird benchmark results.

@dagurtomas dagurtomas removed the awaiting-author A reviewer has asked the author a question or requested changes label Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-category-theory Category theory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants