[Merged by Bors] - fix: add assumption to autoparam isBoundedDefault#14617
[Merged by Bors] - fix: add assumption to autoparam isBoundedDefault#14617
assumption to autoparam isBoundedDefault#14617Conversation
PR summary 8358236561Import 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/no_lost_declarations.sh short <optional_commit>
## more verbose report:
./scripts/no_lost_declarations.sh <optional_commit> |
|
Since it seems to improve the situation, let's get this in! |
|
I am not completely confident in the third test case.
already worked with the previous version. What failed was
which required two more arguments. |
Maybe, `applyc` in lean3 also tried `assumption`, so I added it here! [Zulip thread](https://leanprover.zulipchat.com/#narrow/stream/239415-metaprogramming-.2F-tactics/topic/isBoundedDefault.20not.20triggering)
|
Indeed, apply limsup_le_limsup
exact hstill fails. I've already sent to bors, so the PR will be merged soon, but it doesn't solve the issue. Note that |
|
Yes, I agree. Another workaround is and this reminds something that has changed recently. Let me see if I can find it! |
|
Pull request successfully merged into master. Build succeeded: |
assumption to autoparam isBoundedDefaultassumption to autoparam isBoundedDefault
Maybe,
applycin lean3 also triedassumption, so I added it here!Zulip thread