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

feat: try? tactic improvements #6981

Merged
merged 7 commits into from
Feb 7, 2025
Merged

feat: try? tactic improvements #6981

merged 7 commits into from
Feb 7, 2025

Conversation

leodemoura
Copy link
Member

This PR adds new configuration options to try?.

  • try? -only omits simp only and grind only suggestions
  • try? +missing enables partial solutions where some subgoals are "solved" using sorry, and must be manually proved by the user.
  • try? (max:=<num>) sets the maximum number of suggestions produced (default is 8).

@leodemoura leodemoura added the changelog-language Language features, tactics, and metaprograms label Feb 7, 2025
@leodemoura leodemoura requested a review from kim-em as a code owner February 7, 2025 01:08
@leodemoura leodemoura enabled auto-merge February 7, 2025 01:09
@github-actions github-actions bot temporarily deployed to lean-lang.org/lean4/doc February 7, 2025 01:15 Inactive
@leodemoura leodemoura added this pull request to the merge queue Feb 7, 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 Feb 7, 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 c92425f98de92a3afc4dfc7b4c0ad754fb19e6f4 --onto b01ca8ee237a1b3c299384e73ad79d424e216a84. (2025-02-07 01:39:40)

/-- Maximum number of suggestions. -/
max := 8
/-- If `missing` is `true`, allows the construction of partial solutions where some of the subgoals are `sorry`. -/
missing := false
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+partial (or +sorry) seem clearer than +missing.

Merged via the queue into master with commit 0376cae Feb 7, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

3 participants