Skip to content
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

Reorganize #191

Merged
merged 19 commits into from
Nov 24, 2023
Merged

Reorganize #191

merged 19 commits into from
Nov 24, 2023

Conversation

SimonGuilloud
Copy link
Collaborator

Reafactor and reorganize files:

  • Root project moved to lisa-sets
  • Removed unused legacy files and tests
  • Put all automation tools in the same place
  • Generally flatten folder structure, when possible
  • Clean some files and imports

SimonGuilloud and others added 18 commits November 9, 2023 01:51
 - Multiple theorems in one file.
 - Finish justification mappings.
 - Recreate theorems.
 - Optionally recompute theorems and cache, recompute but not cache, or use cache (sbt tasks?).
 - Evaluate.
… including some from the library with imports.

theorems get fully reconstructed. Now, integrate.
… Can now create a high level THM directly from a kernel theorem.

Left to do: Apply that to proof files. Options to run with cache, without cache without creating it and without cache and create it.
Need to detect changes in statements. POssibly, do hashconsing before storing
…whether serialization is activated to constructors? How to construct theorems in a good order?
…y successful. Running Recursion.scala (i.e. everything) takes around 19s instead of 24s
@sankalpgambhir
Copy link
Member

Most changes look good to me.

I would personally prefer lisa.mathematics over lisa.maths, but it's fine either way.

Can merge after the compilation is fixed.

@SimonGuilloud SimonGuilloud merged commit 1411eac into epfl-lara:main Nov 24, 2023
1 check passed
@SimonGuilloud SimonGuilloud deleted the reorganize branch December 3, 2023 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants