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] - refactor: move natural isomorphisms involving inverses of equivalences #10278

Closed
wants to merge 3 commits into from

Conversation

TwoFX
Copy link
Member

@TwoFX TwoFX commented Feb 5, 2024

  • Move all results from CategoryTheory/Functor/InvIsos.lean to CategoryTheory/Equivalence.lean and delete the former file
  • Replace use of eqToIso (Functor.comp_id G) with G.rightUnitor
  • Move into Iso namespace to enable dot notation
  • Add analogous results stated in terms of Equivalence instead of IsEquivalence

I have to admit that I'm not up to date on the discussion regarding renamings and deprecations. Do I have to add some kind of align statement or deprecation notice due to moving the definitions from Mathlib.CategoryTheory to Mathlib.CategoryTheory.Iso?

Open in Gitpod

@TwoFX TwoFX added awaiting-review The author would like community review of the PR t-category-theory Category theory labels Feb 5, 2024
@joelriou
Copy link
Collaborator

joelriou commented Feb 6, 2024

It seems that the syntax for deprecation would be like:

@[deprecated] -- since 2024-02-06
alias compInvIso := Iso.compInvIso

@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 Feb 6, 2024
@TwoFX TwoFX 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 Feb 7, 2024
@joelriou
Copy link
Collaborator

joelriou commented Feb 7, 2024

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 Feb 7, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 7, 2024
#10278)

- Move all results from `CategoryTheory/Functor/InvIsos.lean` to `CategoryTheory/Equivalence.lean` and delete the former file
- Replace use of `eqToIso (Functor.comp_id G)` with `G.rightUnitor`
- Move into `Iso` namespace to enable dot notation
- Add analogous results stated in terms of `Equivalence` instead of `IsEquivalence`
@mathlib-bors
Copy link

mathlib-bors bot commented Feb 7, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title refactor: move natural isomorphisms involving inverses of equivalences [Merged by Bors] - refactor: move natural isomorphisms involving inverses of equivalences Feb 7, 2024
@mathlib-bors mathlib-bors bot closed this Feb 7, 2024
@mathlib-bors mathlib-bors bot deleted the compinviso branch February 7, 2024 10:52
atarnoam pushed a commit that referenced this pull request Feb 9, 2024
#10278)

- Move all results from `CategoryTheory/Functor/InvIsos.lean` to `CategoryTheory/Equivalence.lean` and delete the former file
- Replace use of `eqToIso (Functor.comp_id G)` with `G.rightUnitor`
- Move into `Iso` namespace to enable dot notation
- Add analogous results stated in terms of `Equivalence` instead of `IsEquivalence`
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