-
Notifications
You must be signed in to change notification settings - Fork 11
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: support fixed integers and conditions in meta-simp-set #346
Conversation
This is helpful for debugging, when we copy the individual statements to the actual proof, which may not have Ctxt open.
Co-authored-by: Siddharth <[email protected]>
Co-authored-by: Siddharth <[email protected]>
…ean-up and extract general theorems
This is work in progress. It seems toSnoc_toMap does not work with nested snocs? @[simp]
theorem toSnoc_toMap {Γ : Ctxt Ty} {t : Ty } {var : Ctxt.Var Γ t} {f : Ty → Ty2} :
@Var.toMap _ (Γ.snoc t) _ _ f var.toSnoc = var.toMap.toSnoc := rfl |
Alive Statistics: 54 / 93 (39 failed) |
Alive Statistics: 54 / 93 (39 failed) |
Alive Statistics: 54 / 93 (39 failed) |
Alive Statistics: 54 / 93 (39 failed) |
LGTM, I am happy to merge this in, with @alexkeizer 's review on it. |
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.
Some minor style comments. I am curious what brought up the decision to change the normal form of ConcreteOrMVar.concrete
, was it just to reduce the noise in goal statements?
Alive Statistics: 54 / 93 (39 failed) |
1 similar comment
Alive Statistics: 54 / 93 (39 failed) |
From my perspective all comments are adressed. @alexkeizer @bollu, could you take a final look? |
Alive Statistics: 54 / 93 (39 failed) |
Reviewed in person by @bollu |
Alive Statistics: 54 / 93 (39 failed) |
ConcreteOrMVar.instantiate_ofNat_eq
to simplify instantiation and canonicalize to ofNat.Unfortunately, this seems to lead to a 'motive not type correct' error. To resolve it we use simp! that just unfolds. This should be debugged further: #349.
toSnoc_toMap
to work on sequences of snoc