Skip to content
This repository has been archived by the owner on Oct 25, 2023. It is now read-only.

chore: don't rely on simp calling decide #168

Merged
merged 2 commits into from
Mar 28, 2023

Conversation

kim-em
Copy link
Contributor

@kim-em kim-em commented Mar 28, 2023

I am trying to change the default behaviour in simp from { decide := true } to { decide := false }.

See

I think I've got all the patches to Lean4 required, except that I need to make a simultaneous change to lake in order to pass CI! (Oh what a tangled web we weave...)

This simple PR makes Lake agnostic to the default behaviour in Lean, compiling successfully either before or after the change. If it is okay to merge this, then hopefully I can proceed with the rest.

@gebner
Copy link
Member

gebner commented Mar 28, 2023

If this fails here, then we're probably missing a ¬ false = true or ¬ true = false lemma in core.

@kim-em
Copy link
Contributor Author

kim-em commented Mar 28, 2023

If this fails here, then we're probably missing a ¬ false = true or ¬ true = false lemma in core.

Ah, good point. I will go hunting.

@tydeu tydeu added the chore Maintenance tasks or refactors label Mar 28, 2023
@tydeu tydeu merged commit 6de8ee8 into leanprover:master Mar 28, 2023
@kim-em
Copy link
Contributor Author

kim-em commented Apr 3, 2023

If this fails here, then we're probably missing a ¬ false = true or ¬ true = false lemma in core.

Ah, good point. I will go hunting.

leanprover/lean4#2182

Kha pushed a commit to Kha/lean4 that referenced this pull request Jul 17, 2023
Kha pushed a commit to Kha/lean4 that referenced this pull request Jul 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
chore Maintenance tasks or refactors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants