Skip to content
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

[Backport releases/v4.18.0] fix: WellFounded preprocessing: use dsimp #7422

Merged
merged 1 commit into from
Mar 10, 2025

Conversation

github-actions[bot]
Copy link
Contributor

Backport c797525 from #7409.

This PR allows the use of `dsimp` during preprocessing of well-founded
definitions. This fixes regressions when using `if-then-else` without
giving a name to the condition, but where the condition is needed for
the termination proof, in cases where that subexpression is reachable
only by dsimp, but not by simp (e.g. inside a dependent let)

Also fixes some preprocessing lemmas to not be bad simp lemmas (with
lambdas on the LHS, due to dot notation and unfortunate argument order)

This fixes #7408.

(cherry picked from commit c797525)
@nomeata
Copy link
Collaborator

nomeata commented Mar 10, 2025

No CI running here?

@nomeata nomeata closed this Mar 10, 2025
@nomeata nomeata reopened this Mar 10, 2025
@nomeata
Copy link
Collaborator

nomeata commented Mar 10, 2025

Ah, classic issue with bot created PRs. Reopening helps.

@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc March 10, 2025 17:37 Inactive
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Mar 10, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Mar 10, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Mar 10, 2025
@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Mar 10, 2025
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

@nomeata nomeata merged commit efb8ae7 into releases/v4.18.0 Mar 10, 2025
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builds-mathlib CI has verified that Mathlib builds against this PR toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants