Fix definition dependency & Sort jsonl #8
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
My previous commit (#6) didn't add the definition of the theorem dependencies. I've fixed it now. Sorry, it was a serious mistake.
PR #7 points out that the jsonl file might be out of order and there was a missing log file for the script. I’ve added sorting to the script so the jsonl entries are now arranged in lexicographic order. I also added the log file to
.gitignore, so we don’t have to worry about it anymore.There are two other things I want to discuss:
The ProofNetSharp project splits the dataset into valid and test sets, similar to MiniF2F. If users want to make use of that split, they can try this project. However, ProofNetSharp is missing
Cambridge-Tripos.lean.I saw that some statements declared in
instance. In fact, they can all be converted into theorems completely equivalently by adding a predicateNonempty. Here is an example fromDummit-Foote.lean:Even the proofs are exactly the same. I'm not sure if we need to unify all the data into
theorem.