Skip to content

fix: kernel type mismatch in cutsat eq_def proofs#13587

Open
leodemoura wants to merge 1 commit intomasterfrom
grind_lia_bad_proof
Open

fix: kernel type mismatch in cutsat eq_def proofs#13587
leodemoura wants to merge 1 commit intomasterfrom
grind_lia_bad_proof

Conversation

@leodemoura
Copy link
Copy Markdown
Member

This PR fixes a kernel type mismatch raised by lia/grind when internalizing an integer expression whose syntactic structure differs from the structure of its polynomial representation. The mismatch occurred because the eq_def proof term bridged x.denote ctx = e.denote ctx to Poly.denote' ctx p = 0 via a plain Eq.refl e, but Poly.denote' collapses sub-structure such as a trailing + 0 (the (.num 0) monomial is dropped) while e keeps it. The kernel then rejected the application because the equality between x.denote and Poly.denote' p did not hold definitionally.

Closes #13572.

@leodemoura leodemoura requested a review from kim-em as a code owner April 30, 2026 15:16
@leodemoura leodemoura added the changelog-tactics User facing tactics label Apr 30, 2026
@leodemoura leodemoura enabled auto-merge April 30, 2026 15:16
@leodemoura leodemoura added this pull request to the merge queue Apr 30, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 30, 2026
@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 Apr 30, 2026
@mathlib-lean-pr-testing
Copy link
Copy Markdown

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 19baa470e58e04c5832ef7100e9f5de57d9a7704 --onto 0a6c31520b2ebaa6e71227454b46ebfb4986ae7b. You can force Mathlib CI using the force-mathlib-ci label. (2026-04-30 16:15:01)

@leanprover-bot
Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase 19baa470e58e04c5832ef7100e9f5de57d9a7704 --onto 3fc99eef102549c743c6c63547983319bfae6f01. You can force reference manual CI using the force-manual-ci label. (2026-04-30 16:15:04)

@leodemoura leodemoura enabled auto-merge May 1, 2026 00:42
@leodemoura leodemoura force-pushed the grind_lia_bad_proof branch from 8b2bd5a to 4033d00 Compare May 1, 2026 00:44
@leodemoura leodemoura force-pushed the grind_lia_bad_proof branch from 4033d00 to afbfa6c Compare May 1, 2026 00:46
@leodemoura leodemoura added this pull request to the merge queue May 1, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks May 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-tactics User facing tactics 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.

lia fails with "(kernel) application type mismatch"

2 participants