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.

Sign up for GitHub

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

Jump to bottom

fix: offset terms internalization #6777

Merged
merged 4 commits into from
Jan 25, 2025
Merged

fix: offset terms internalization #6777

merged 4 commits into from
Jan 25, 2025

Conversation

leodemoura
Copy link
Member

This PR fixes a bug in the internalization of offset terms in the grind tactic. For example, grind was failing to solve the following example because of this bug.

example (f : Nat → Nat) : f (a + 1) = 1 → a = 0 → f 1 = 1 := by
  grind

@leodemoura leodemoura added the changelog-language Language features, tactics, and metaprograms label Jan 25, 2025
@leodemoura leodemoura enabled auto-merge January 25, 2025 20:57
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc January 25, 2025 21:03 Inactive
@leodemoura leodemoura added this pull request to the merge queue Jan 25, 2025
@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 Jan 25, 2025
@leanprover-community-bot
Copy link
Collaborator

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 9565334c0e5b98fc814e101aba3695f3972ce9be --onto 58c7a4f15e2f91f8fb93687844e97582ba0b9f10. (2025-01-25 21:18:49)

Merged via the queue into master with commit 6dbb54d Jan 25, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
changelog-language Language features, tactics, and metaprograms toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@leodemoura @leanprover-community-bot