-
Notifications
You must be signed in to change notification settings - Fork 5
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
Deaxiomatizing System E #3
Comments
Hi, we probably won't have the capacity for this, but PRs are welcome! |
Hi, for mathlib I remember that they wanted me to create a PR from a branch, in which case I would need to be given some sort of permission. In this case, should I just make my changes in a fork and then create a PR from that fork? |
You could just PR from your own fork. |
alright thanks! |
The proofs in Book/Prop01 are failing cause euclid_apply can't solve the goals. I also noticed that some of the latest commits are failing the ci process. Is this just an issue with the latest commits? |
Which theorem is failing? Can you paste the SMT query displayed in the infoview panel? Most failures are due to the instability of SMT solvers across different platforms or even different runs on the same platform. It may help to upgrade to the latest development version of Z3 and CVC5. |
It's failing on this theorem
It's saying could not prove: ¬b = a |
If it's Can you take a screenshot of the infoview panel? |
somehow it's not picking up on symmetry of equality or something |
It would be more helpful if the screenshot include complete information. Currently I cannot see which tactic is failing and the exact error message. |
I'm not able to reproduce this issue. @loganrjmurphy any idea? |
Sorry folks, I've just returned from a few weeks away. I'll look into this this week. |
Hi Frederick, I also can't reproduce what you're seeing. Could you confirm which version of Lean is being used? I.e., from within the project directory, run |
sorry, I just got a new computer after the old one bricked. I'm trying to setup the repo again i cant seem to find |
Hello. I'm having the same issue after cloning and running the instructions. NOTE: to get it to build I did have to change Solve.lean /- invoke `applyHole` only if the hole actually exists -/
def finish : EuclidStepM Unit := do
let Γ ← read
withMainContext do
match (← getLCtx).findFromUserName? Γ.holeName with
| none => dbg_trace "finish: fail" ; failure
| some d =>
applyHole (← d.toExpr.toSyntax) (← getUnusedUserName `h) |>.run Γ
evalTactic $ ← `(tactic| try (clear ($(mkIdent Γ.holeName))))
evalTactic $ ← `(tactic| try aesop (add safe [destructProducts])) /- <- Changed line 70 -/ smt-portfolio & Co is installed PS C:\Users\suyas\OneDrive\Documents\LeanEuclid> Where.exe smt-portfolio
C:\Users\suyas\OneDrive\Documents\LeanEuclid\.venv\Scripts\smt-portfolio.exe
PS C:\Users\suyas\OneDrive\Documents\LeanEuclid> lake script run check
All requirements are satisfied. |
Right now System E is formulated as a bunch of axioms.
However, I believe it would more sensible for it to be formulated as a type class such as Group.
All of the theorems would more or less be the same, but proofs from LeanEuclid could more readily be used to prove theorems about the EuclideanSpace typeclass.
Additionally, it would make it easier to construct models of System E and prove the consistency of System E and/or that it has been properly formalized.
I understand that this may be beyond the scope of the project, but I would be willing to do this in a separate branch and make a pull request.
The text was updated successfully, but these errors were encountered: