-
Notifications
You must be signed in to change notification settings - Fork 381
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
chore: make map_ofNat a simp lemma #19388
Conversation
eric-wieser
commented
Nov 22, 2024
PR summary 28e92f2640Import changes for modified filesNo significant changes to the import graph Import changes for all files
Declarations diffNo declarations were harmed in the making of this PR! 🐙 You can run this locally as follows## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>
## more verbose report:
./scripts/declarations_diff.sh long <optional_commit> The doc-module for No changes to technical debt.You can run this locally as
|
!bench |
Here are the benchmark results for commit 28e92f2. Benchmark Metric Change
=================================================================
- ~Mathlib.Algebra.Category.ModuleCat.Basic instructions 49.8% |
3 files, Instructions +3.0⬝10⁹
12 files, Instructions +2.0⬝10⁹
57 files, Instructions +1.0⬝10⁹
|
What is this lemma meant to index on? It's discrimination tree key must be terrible. |
Ideally it would index on |
Done in #19410 |
See the benchmark in #19388, which while not disastrous is not ideal either. Unless the situation changes with leanprover/lean4#2867, the DiscrTree key would apply to all funlike operations applied to any argument, not just applied to numerals.
If I understand correctly this should be closed now. Please reopen if that's incorrect. |