-
Notifications
You must be signed in to change notification settings - Fork 251
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] - perf(AlgebraicGeometry): Fix slow and bad proofs #7747
Conversation
Note: I have pushed an update to the lean toolchain because this PR was on a buggy version of the toolchain. WARNING: checking out old commits of this PR using v4.2.0-rc2 or v4.2.0-rc3 can cause |
@erdOne This seems like an improvement! Would you mind fixing the build please? |
@erdOne Feel free to revert my hacky 42a9855 as part of this (much better) PR, or ping me if you want me to do the revert instead. I didn't know of this PR before. Edit: I ended up just reverting it myself to save you the trouble, hopefully that's not a problem. Feel free to force push your local copy if it conflicts with any changes you might have. |
@jcommelin The build failure was just a "no space left on device error". I triggered a rebuild and it was green. |
@collares Thanks for the fix! |
!bench |
Here are the benchmark results for commit 55c54de. Benchmark Metric Change
==============================================================================
+ ~Mathlib.AlgebraicGeometry.AffineScheme instructions -84.4%
- ~Mathlib.AlgebraicGeometry.GammaSpecAdjunction instructions 8.5%
+ ~Mathlib.AlgebraicGeometry.Morphisms.QuasiSeparated instructions -75.3%
+ ~Mathlib.AlgebraicGeometry.Morphisms.RingHomProperties instructions -24.2%
- ~Mathlib.AlgebraicGeometry.Restrict instructions 5.1%
- ~Mathlib.Topology.Sheaves.SheafCondition.UniqueGluing instructions 51.0% (Note: the three files becoming slower is due to new lemmas added to those files. - Junyan) |
bors merge |
Fixed `AlgebraicGeometry/AffineSchemes.lean`, `AlgebraicGeometry/Morphisms/QuasiSeparated.lean` and `AlgebraicGeometry/Morphisms/RingHomProperties.lean`. Co-authored-by: Mario Carneiro <di.gama@gmail.com> Co-authored-by: Andrew Yang <36414270+erdOne@users.noreply.github.com> Co-authored-by: Johan Commelin <johan@commelin.net> Co-authored-by: Mauricio Collares <mauricio@collares.org>
Pull request successfully merged into master. Build succeeded: |
Fixed `AlgebraicGeometry/AffineSchemes.lean`, `AlgebraicGeometry/Morphisms/QuasiSeparated.lean` and `AlgebraicGeometry/Morphisms/RingHomProperties.lean`. Co-authored-by: Mario Carneiro <di.gama@gmail.com> Co-authored-by: Andrew Yang <36414270+erdOne@users.noreply.github.com> Co-authored-by: Johan Commelin <johan@commelin.net> Co-authored-by: Mauricio Collares <mauricio@collares.org>
Fixed `AlgebraicGeometry/AffineSchemes.lean`, `AlgebraicGeometry/Morphisms/QuasiSeparated.lean` and `AlgebraicGeometry/Morphisms/RingHomProperties.lean`. Co-authored-by: Mario Carneiro <di.gama@gmail.com> Co-authored-by: Andrew Yang <36414270+erdOne@users.noreply.github.com> Co-authored-by: Johan Commelin <johan@commelin.net> Co-authored-by: Mauricio Collares <mauricio@collares.org>
Fixed `AlgebraicGeometry/AffineSchemes.lean`, `AlgebraicGeometry/Morphisms/QuasiSeparated.lean` and `AlgebraicGeometry/Morphisms/RingHomProperties.lean`. Co-authored-by: Mario Carneiro <di.gama@gmail.com> Co-authored-by: Andrew Yang <36414270+erdOne@users.noreply.github.com> Co-authored-by: Johan Commelin <johan@commelin.net> Co-authored-by: Mauricio Collares <mauricio@collares.org>
Fixed
AlgebraicGeometry/AffineSchemes.lean
,AlgebraicGeometry/Morphisms/QuasiSeparated.lean
andAlgebraicGeometry/Morphisms/RingHomProperties.lean
.