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

Tactics should signal wrong ProofInfo #421

Open
Engreyight opened this issue Jan 16, 2024 · 1 comment
Open

Tactics should signal wrong ProofInfo #421

Engreyight opened this issue Jan 16, 2024 · 1 comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement Enhancement of existing features

Comments

@Engreyight
Copy link
Collaborator

Certain tactics, such as mlRewriteBy only work if the goal is using AnyReasoning. However, if they are used with something different, they fail with a useless error message (mlRewriteBy says "not in proof mode"). Instead, they should signal that the ProofInfo is incorrect, or add an extra ProofInfoLE AnyReasoning i goal, like mlApplyMeta does in similar situations.

This limitation is also undocumented.

@Engreyight Engreyight added bug Something isn't working documentation Improvements or additions to documentation enhancement Enhancement of existing features labels Jan 16, 2024
@berpeti
Copy link
Collaborator

berpeti commented Feb 1, 2024

There are two subtasks here:

  • Update documentation.
  • Fix error messages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement Enhancement of existing features
Projects
None yet
Development

No branches or pull requests

2 participants