-
Notifications
You must be signed in to change notification settings - Fork 259
[ add ] clean version of Data.Fin.Properties.searchMinimalCounterexample
#2801
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like all of this, except for the name μ. It's a cute pun, but caters to a rather small crowd.
How about calling the constructor least instead? If you want a pun, then μ⟨_⟩ can be called the-least. I don't think μ⟨¬_⟩ passes the Fairbairn threshold.
It was absolutely not intended as a pun:
Well...
|
|
@JacquesCarette You were right, and I was wrong! |
I somehow never tire of hearing that 😏 . I'm quite happy with the current results. |
Revised version of the relevant code from #2744 incorporating: UPDATED
MinimalCounterexample/Least⟨¬_⟩Least⟨_⟩now defined as arecord, not assyntaxinfix versions now as (duplicate) definitions of theremoved verbose CamelCase versionsinjectdeprecated in favour of_<_; addinject-<as lemma to smooth the pathOutstanding issues, if this PR gets merged:
¬∀⟶∃¬-smallestand¬∀⟶∃¬should be deprecated, as superseded bysearch-least⟨¬_⟩?¬∀⇒∃¬-smallestand¬∀⇒∃¬, cf.¬∃⇒∀¬#2831 etc. UPDATED: see [ deprecate ] names for deMorgan lemmas inData.Fin.Properties#2837