-
Notifications
You must be signed in to change notification settings - Fork 354
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(Algebra/Homology): use the new homology API #8706
Conversation
This part of you message seems to have swapped some lefts and rights. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
bors d+
✌️ joelriou can now approve this pull request. To approve and merge a pull request, simply reply with |
Co-authored-by: Johan Commelin <johan@commelin.net>
Thanks very much @jcommelin for all these reviews! bors merge |
This PR refactors the construction of left derived functors using the new homology API: this also affects the dependencies (Ext functors, group cohomology, local cohomology). As a result, the old homology API is no longer used in any significant way in mathlib. Then, with this PR, the homology refactor is essentially complete. The organization of the files was made more coherent: the definition of a projective resolution is in `Preadditive.ProjectiveResolution`, the existence of resolutions when there are enough projectives is shown in `Abelian.ProjectiveResolution`, and the left derived functor is constructed in `Abelian.LeftDerived`; the dual results are in `Preadditive.InjectiveResolution`, `Abelian.InjectiveResolution` and `Abelian.RightDerived`. Co-authored-by: Joël Riou <37772949+joelriou@users.noreply.github.com>
Pull request successfully merged into master. Build succeeded: |
This PR refactors the construction of left derived functors using the new homology API: this also affects the dependencies (Ext functors, group cohomology, local cohomology). As a result, the old homology API is no longer used in any significant way in mathlib. Then, with this PR, the homology refactor is essentially complete. The organization of the files was made more coherent: the definition of a projective resolution is in `Preadditive.ProjectiveResolution`, the existence of resolutions when there are enough projectives is shown in `Abelian.ProjectiveResolution`, and the left derived functor is constructed in `Abelian.LeftDerived`; the dual results are in `Preadditive.InjectiveResolution`, `Abelian.InjectiveResolution` and `Abelian.RightDerived`. Co-authored-by: Joël Riou <37772949+joelriou@users.noreply.github.com>
This PR refactors the construction of left derived functors using the new homology API: this also affects the dependencies (Ext functors, group cohomology, local cohomology). As a result, the old homology API is no longer used in any significant way in mathlib. Then, with this PR, the homology refactor is essentially complete.
The organization of the files was made more coherent: the definition of a projective resolution is in
Preadditive.ProjectiveResolution
, the existence of resolutions when there are enough projectives is shown inAbelian.ProjectiveResolution
, and the left derived functor is constructed inAbelian.LeftDerived
; the dual results are inPreadditive.InjectiveResolution
,Abelian.InjectiveResolution
andAbelian.RightDerived
.