Skip to content

fix: apply beta when canonicalizing types in grind - #14709

Merged
leodemoura merged 2 commits into
masterfrom
sym_canon_beta_types
Aug 7, 2026
Merged

fix: apply beta when canonicalizing types in grind#14709
leodemoura merged 2 commits into
masterfrom
sym_canon_beta_types

Conversation

@leodemoura

Copy link
Copy Markdown
Member

This PR ensures beta-reduction is applied when canonicalizing types in grind.

This PR ensures beta-reduction is applied when canonicalizing types in `grind`.
@leodemoura leodemoura added the changelog-tactics User facing tactics label Aug 6, 2026
@leodemoura
leodemoura enabled auto-merge August 6, 2026 20:40
@leodemoura
leodemoura disabled auto-merge August 6, 2026 22:45
@leodemoura
leodemoura enabled auto-merge August 6, 2026 22:45
kim-em added a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Aug 7, 2026
Repairs 22 `grind` calls across 11 files. Three are hint-list changes only
(`Perm/Fin`, `Gaps`, `DeltaZeroIter`); the rest supply a `Fin.val` fact or
finite case analysis that the `ToInt` machinery used to put in the E-graph.

`#adaptation_note`s mark the genuine workarounds, and flag the four sites
whose diagnostics suggest leanprover/lean4#14709 (grind conflating two
spellings of the same `Fin` type) rather than a missing `[grind hom]` rule.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CM7yQ5HjNMiMucvCud9RdT
@leodemoura
leodemoura disabled auto-merge August 7, 2026 12:36
@leodemoura
leodemoura enabled auto-merge August 7, 2026 12:36
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@leodemoura
leodemoura added this pull request to the merge queue Aug 7, 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 Aug 7, 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 4a37393b74d177d5e32f06cfd097ff6eb9f507b8 --onto bd97ed8e35639e17a11d1490cbc46d994e7b4513. You can force Mathlib CI using the force-mathlib-ci label. (2026-08-07 13:08:03)

@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 4a37393b74d177d5e32f06cfd097ff6eb9f507b8 --onto c4e6b62c3d955ef20da94310797072f7c4c5fa2b. You can force reference manual CI using the force-manual-ci label. (2026-08-07 13:08:05)

Merged via the queue into master with commit 10d40ba Aug 7, 2026
19 checks passed
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.

2 participants