-
Notifications
You must be signed in to change notification settings - Fork 234
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 (Functor.Flat): restructure proof to reduce simp time #8065
Conversation
bench! |
!bench |
Here are the benchmark results for commit 1613e95. Benchmark Metric Change
=================================================================
- ~Mathlib.Analysis.NormedSpace.Multilinear instructions 1.6%
+ ~Mathlib.CategoryTheory.Functor.Flat instructions -56.7% |
bors merge |
This file recently jumped almost 200% in #8051. Changing a `simpa using X` call into a `convert X <;> simp` cuts the clock time for the file in half.
bors merge- |
Canceled. |
Sorry there is a line I commented out that I should either delete or uncomment |
@j-loreaux if you want to pull the trigger again, please do. 🙏 |
bors merge |
This file recently jumped almost 200% in #8051. Changing a `simpa using X` call into a `convert X <;> simp` cuts the clock time for the file in half.
Pull request successfully merged into master. Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
This file recently jumped almost 200% in #8051. Changing a `simpa using X` call into a `convert X <;> simp` cuts the clock time for the file in half.
This file recently jumped almost 200% in #8051. Changing a
simpa using X
call into aconvert X <;> simp
cuts the clock time for the file in half.