-
Notifications
You must be signed in to change notification settings - Fork 364
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
Create a backbone of Defs.lean
files
#19253
Comments
Here's an overview of existing Clean Defs files(including open PRs, as of 2024-11-15)
Nearly clean defs
These files are not definitions-only:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The import hierarchy in Mathlib is quite messy. A concrete issue we can focus on is that lot of files in Mathlib are structured around introducing a definition and then following up with a large collection of lemmas. This means that importing a definition often means also pulling in many unrelated definitions that interact with the intended result. Not only does this clustering make Mathlib slower to build after changes, it also means harder minimization of examples.
Ideally we'd maintain a stable "backbone" of
Defs
files, where each only contains definitions and the bare minimum of theory to get to the next level in the backbone. These could then be paired withLemmas
files that can be imported to get the full complement of results that would currently be imported alongside the defs.Studying the import graph for a file can be a very useful tool in figuring out which bits of theory should be unnecessary for a definition.
The text was updated successfully, but these errors were encountered: