Skip to content

feat: revamp aux decl name generation#8363

Merged
Kha merged 2 commits intoleanprover:masterfrom
Kha:push-vlqyznwmtuzv
May 16, 2025
Merged

feat: revamp aux decl name generation#8363
Kha merged 2 commits intoleanprover:masterfrom
Kha:push-vlqyznwmtuzv

Conversation

@Kha
Copy link
Copy Markdown
Member

@Kha Kha commented May 15, 2025

This PR unifies various ways of naming auxiliary declarations in a conflict-free way and ensures the method is compatible with diverging branches of elaboration such as parallelism or Aesop-like backtracking+replaying search.

@Kha Kha requested review from leodemoura and zwarich as code owners May 15, 2025 15:28
@Kha Kha added awaiting-mathlib We should not merge this until we have a successful Mathlib build changelog-language Language features and metaprograms labels May 15, 2025
@Kha Kha force-pushed the push-vlqyznwmtuzv branch from ca5c3f4 to 23e42a5 Compare May 15, 2025 15:31
@Kha Kha force-pushed the push-vlqyznwmtuzv branch from 23e42a5 to 978b399 Compare May 16, 2025 08:38
@Kha Kha requested a review from kim-em as a code owner May 16, 2025 08:38
@Kha Kha force-pushed the push-vlqyznwmtuzv branch 2 times, most recently from eba1110 to 73c2555 Compare May 16, 2025 08:41
@Kha
Copy link
Copy Markdown
Member Author

Kha commented May 16, 2025

@JLimperg Could you test whether this can be used to resolve the aesop+omega issue? You'd need to use mkChild for independent search branches like in wrapAsync: https://github.com/leanprover/lean4/pull/8363/files#diff-4b46569e47cd3ab819703ca7627d137c2768803fe1ac84ef56e2782c8f23bd58R532-R535

@Kha Kha force-pushed the push-vlqyznwmtuzv branch from 73c2555 to cb0d344 Compare May 16, 2025 09:08
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label May 16, 2025
ghost pushed a commit to leanprover-community/batteries that referenced this pull request May 16, 2025
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 16, 2025
@ghost
Copy link
Copy Markdown

ghost commented May 16, 2025

@ghost ghost added the breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan label May 16, 2025
@ghost
Copy link
Copy Markdown

ghost commented May 16, 2025

@Kha Kha force-pushed the push-vlqyznwmtuzv branch from cb0d344 to e87bdca Compare May 16, 2025 12:02
@Kha Kha force-pushed the push-vlqyznwmtuzv branch from e87bdca to db9ce34 Compare May 16, 2025 12:06
ghost pushed a commit to leanprover-community/batteries that referenced this pull request May 16, 2025
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request May 16, 2025
@ghost
Copy link
Copy Markdown

ghost commented May 16, 2025

@ghost
Copy link
Copy Markdown

ghost commented May 16, 2025

@ghost ghost added builds-mathlib CI has verified that Mathlib builds against this PR and removed awaiting-mathlib We should not merge this until we have a successful Mathlib build breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan labels May 16, 2025
@ghost
Copy link
Copy Markdown

ghost commented May 16, 2025

@Kha Kha added this pull request to the merge queue May 16, 2025
Merged via the queue into leanprover:master with commit 4d58a3d May 16, 2025
22 of 23 checks passed
@Kha Kha deleted the push-vlqyznwmtuzv branch May 16, 2025 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-mathlib CI has verified that Mathlib builds against this PR changelog-language Language features and metaprograms toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants