-
Notifications
You must be signed in to change notification settings - Fork 66
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
DATA: Create an automated tool to regularly gather the equations from Equations.lean and AllEquations.lean and output them to a text file that is easy for machines to read #142
Comments
Currently #126 only considers equations that appear in some implications, so this will require some change |
It would also be nice to have a separate file that only lists the implications and anti implications on the equations defined just in Equations.lean |
Recently, Equations 28393, 374794, and 5105 have appeared in the output of |
That also broke some of my tools. I could add a Lean script to extract the equations, what would be a good I/O for it? |
If you mean an output format, just a JSON hash seems easy for everyone to consume. Are you thinking of including it in |
I think |
As a short term fix one can of course add these equations in commented form to AllEquations.lean, and also add a note sonewhere in CONTRIBUTING.md requesting that one add the commented equation to AllEquations.lean whenever adding a novel equation to Equation.lean as a courtesy. |
Personally, I'd go for the simpler textual form since lots of tools just need to display them, but there's nothing stopping you from having it be |
claim |
disclaim |
claim |
propose PR #429 |
This PR depends on #427. |
This PR modifies the `equation` command to automatically generate the corresponding law with variables over `Fin n` (in addition to the law with variables over `Nat` that is already being generated). This is used to show that the automatically generated laws are equivalent to the original equation in terms of satisfiability. Additionally, this PR introduces an environment extension to store all the equational laws, which can be used to export laws as described in #142. This PR also modifies a result transporting laws from `Fin n` to `Nat` to omit the `n != 0` hypothesis. --------- Co-authored-by: Pietro Monticone <38562595+pitmonticone@users.noreply.github.com>
I get the following build error on
|
This would presumably already be a byproduct of #29, but is worth isolating separately. The model here is that
Equations.lean
andAllEquations.lean
represent the ground truth of the universe of equations/laws under consideration, so that new equations can be added to the universe by updating the lean files accordinglyThe text was updated successfully, but these errors were encountered: