Skip to content

chore: document two heavy grind uses - #850

Merged
chenson2018 merged 2 commits into
leanprover:mainfrom
jessealama:chore/document-heavy-grinds
Sep 1, 2026
Merged

chore: document two heavy grind uses#850
chenson2018 merged 2 commits into
leanprover:mainfrom
jessealama:chore/document-heavy-grinds

Conversation

@jessealama

Copy link
Copy Markdown
Contributor

Document a couple of heavy grind uses flagged by the weekly mergeWithGrind report.

The suggested merge (dropping the have before the final grind) works,
but roughly triples the file's compile time, so keep the have and
silence the linter with an explanatory comment.
…lint

The suggestion (dropping specialize ih w') only holds for one of the
two goals under all_goals; applying it breaks the proof of the other
goal. Silence the linter with an explanatory comment.

@chenson2018 chenson2018 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very useful to have these documented, thanks!

@chenson2018
chenson2018 added this pull request to the merge queue Sep 1, 2026
Merged via the queue into leanprover:main with commit daedb5e Sep 1, 2026
2 checks passed
ctchou pushed a commit to ctchou/cslib that referenced this pull request Sep 1, 2026
…eanprover#851)

Golf a heavy proof that was recently flagged as such (in leanprover#850) by
splitting `all_goals` into bullets and doing the work.
@jessealama
jessealama deleted the chore/document-heavy-grinds branch September 1, 2026 20:48
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