Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

Conversation

@eric-wieser
Copy link
Member

@eric-wieser eric-wieser commented Oct 14, 2020

@fpvandoorn
Copy link
Member

What happens when you apply this to a conv-goal? Ideally there would be an understandable error message.

@eric-wieser
Copy link
Member Author

Well, exact rfl behaves exactly the same as skip in conv mode.

@urkud
Copy link
Member

urkud commented Oct 15, 2020

Could you please add a test case?

Comment on lines +117 to +118
introduced which are impossible to close with `conv` tactics. Note that goals like `⊢ a = 0` will
be displayed by conv mode as `| a`, making the problem difficult to diagnose at times.
Copy link
Member

Choose a reason for hiding this comment

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

Is this fixable, by checking whether the RHS is a metavariable when displaying the goal?

Copy link
Member Author

@eric-wieser eric-wieser Oct 15, 2020

Choose a reason for hiding this comment

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

I think the fix would have to be within lean itself. That would certainly be better than what happens today, but it doesn't strike me as exhaustive.

@fpvandoorn
Copy link
Member

Ah, ok, so usually a goal | t means t = ?m? That would also be useful to note in the docstring of the exact tactic. Something like "The goal | t means t = ?m, which means that you can use the tactic exact h whenever h has type t = s."

@robertylewis
Copy link
Member

I second the requests for tests and additional doc comments. This looks fine to me as a stopgap. Longer term I think we should reexamine the conv implementation and documentation. I find it inconvenient and annoying so I mostly try to avoid it myself.

@robertylewis robertylewis added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Oct 18, 2020
@YaelDillies
Copy link
Collaborator

This is now a different problem in Lean 4 (does it need solving?). To be revisited after the port.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

awaiting-author A reviewer has asked the author a question or requested changes maybe-later t-meta Tactics, attributes or user commands

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants