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

[Merged by Bors] - feat(RepresentationTheory/Action): add more limit properties of Action V G and Action.forget V G #9603

Closed
wants to merge 5 commits into from

Conversation

chrisflav
Copy link
Collaborator

Adds instances on limit properties of Action V G and Action.forget V G.


Open in Gitpod

@chrisflav chrisflav added awaiting-review The author would like community review of the PR awaiting-CI t-category-theory Category theory labels Jan 10, 2024
Copy link
Collaborator

@joelriou joelriou left a comment

Choose a reason for hiding this comment

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

I have left some comments about small improvements.

@@ -40,6 +40,11 @@ instance [HasFiniteLimits V] : HasFiniteLimits (Action V G) where
instance [HasLimits V] : HasLimits (Action V G) :=
Adjunction.has_limits_of_equivalence (Action.functorCategoryEquivalence _ _).functor

/-- If `V` has limits of shape `J`, so does `Action V G`.-/
lemma hasLimitsOfShapeOfHasLimitsOfShape {J : Type w₁} [Category.{w₂} J] [HasLimitsOfShape J V] :
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this can be made an instance (with a slightly simpler name), and similarly for colimits:

Suggested change
lemma hasLimitsOfShapeOfHasLimitsOfShape {J : Type w₁} [Category.{w₂} J] [HasLimitsOfShape J V] :
instance hasLimitsOfShape {J : Type w₁} [Category.{w₂} J] [HasLimitsOfShape J V] :

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This was an instance originally (see the first commit in this PR), but it caused a instance synthesization timeout in Mathlib.RepresentationTheory.FdRep at HasKernels (FdRep k G) (see: https://github.com/leanprover-community/mathlib4/actions/runs/7468254528/job/20323323137). Bumping the maximum number of heartbeats there to 50000 fixed it, but I was not sure whether this is desirable. Maybe this can be circumvented by adding some more auxiliary instances in the FdRep file to make the instance search faster?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I have tried to investigate this. I have just pushed this 8c5c763 which seems to fix the problem.

Mathlib/RepresentationTheory/Action/Limits.lean Outdated Show resolved Hide resolved
@joelriou joelriou added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jan 11, 2024
@chrisflav chrisflav added awaiting-review The author would like community review of the PR awaiting-CI and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jan 11, 2024
@joelriou
Copy link
Collaborator

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Jan 12, 2024
mathlib-bors bot pushed a commit that referenced this pull request Jan 12, 2024
…on V G` and `Action.forget V G` (#9603)

Adds instances on limit properties of `Action V G` and `Action.forget V G`.



Co-authored-by: Joël Riou <joel.riou@universite-paris-saclay.fr>
@mathlib-bors
Copy link

mathlib-bors bot commented Jan 12, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title feat(RepresentationTheory/Action): add more limit properties of Action V G and Action.forget V G [Merged by Bors] - feat(RepresentationTheory/Action): add more limit properties of Action V G and Action.forget V G Jan 12, 2024
@mathlib-bors mathlib-bors bot closed this Jan 12, 2024
@mathlib-bors mathlib-bors bot deleted the chrisflav/action-preserve-2 branch January 12, 2024 14:03
linesthatinterlace pushed a commit that referenced this pull request Jan 16, 2024
…on V G` and `Action.forget V G` (#9603)

Adds instances on limit properties of `Action V G` and `Action.forget V G`.



Co-authored-by: Joël Riou <joel.riou@universite-paris-saclay.fr>
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.

None yet

2 participants