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] - chore: @[simp] cancel_(right|left) #6300
Conversation
semorrison
commented
Aug 2, 2023
•
edited
edited
- depends on: [Merged by Bors] - chore: remove reducible from Function.Surjective #6297
This PR/issue depends on:
|
dec3762
to
f5ac19e
Compare
f5ac19e
to
d63c234
Compare
!bench |
Here are the benchmark results for commit d63c234. Benchmark Metric Change
=========================================================================================
- ~Mathlib.CategoryTheory.Limits.FilteredColimitCommutesFiniteLimit instructions 5.9% |
One file got slower, but overall wall-clock is identical (epsilon faster), so I propose not worrying. |
I don't understand: how will the simplifier find these injectivity/surjectivity assumptions? |
What is the motivation for this PR? Are you planning to golf some proofs in mathlib? Or some others? |
This was motivated by @jakelev and I working on our So, no, not planning on golfing specific goals, we just wanted Not sure I'm going to have much opportunity to work on |
These can be triggered if Mostly though the huge amount of repetition in the library that this PR highlights makes me wonder if |
@kbuzzard, e.g. If you have an injectivity hypotheses |
Let's get this in. Thanks! |
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
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. |
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>