Skip to content

chore: simp tracing reports ←#2621

Merged
kim-em merged 5 commits intoleanprover:masterfrom
kim-em:simp_trace_inv
Oct 15, 2023
Merged

chore: simp tracing reports ←#2621
kim-em merged 5 commits intoleanprover:masterfrom
kim-em:simp_trace_inv

Conversation

@kim-em
Copy link
Copy Markdown
Collaborator

@kim-em kim-em commented Oct 4, 2023

@kim-em kim-em requested a review from digama0 October 4, 2023 02:42
@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 Oct 4, 2023
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Oct 4, 2023
@ghost ghost added the breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan label Oct 4, 2023
@ghost
Copy link
Copy Markdown

ghost commented Oct 4, 2023

@digama0 digama0 self-assigned this Oct 4, 2023
@kim-em
Copy link
Copy Markdown
Collaborator Author

kim-em commented Oct 4, 2023

@digama0 if it's useful for reviewing I can make proposed downstream changes to Std and Mathlib as well, but won't immediately unless you say so.

@kim-em
Copy link
Copy Markdown
Collaborator Author

kim-em commented Oct 10, 2023

@digama0, I'd like to get this in, but am uncertain whether your self-assignment above means:

  • that I shouldn't do further work on this because you have in mind a better solution?
  • that I should do the downstream fixes to Std and Mathlib required for this approach, and then you'll review?

@digama0
Copy link
Copy Markdown
Collaborator

digama0 commented Oct 10, 2023

It means I was planning on reviewing it, but I was at a conference last week so I'm just now working down my backlog. I'll take a look at it now or tomorrow.

ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Oct 10, 2023
kim-em added a commit to kim-em/std4 that referenced this pull request Oct 13, 2023
kim-em added a commit to kim-em/aesop that referenced this pull request Oct 13, 2023
@kim-em
Copy link
Copy Markdown
Collaborator Author

kim-em commented Oct 13, 2023

ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Oct 13, 2023
kim-em added a commit to leanprover-community/mathlib4 that referenced this pull request Oct 13, 2023
@ghost ghost added builds-mathlib CI has verified that Mathlib builds against this PR and removed breaks-mathlib This is not necessarily a blocker for merging: but there needs to be a plan labels Oct 13, 2023
@kim-em kim-em added the awaiting-review Waiting for someone to review the PR label Oct 14, 2023
@kim-em kim-em requested a review from digama0 October 14, 2023 03:32
@kim-em
Copy link
Copy Markdown
Collaborator Author

kim-em commented Oct 14, 2023

I've checked that downstream projects survive okay. There is some unfortunate copypasta there that ideally we can clean up later.

@kim-em kim-em merged commit 66ab016 into leanprover:master Oct 15, 2023
kim-em added a commit to leanprover-community/batteries that referenced this pull request Oct 16, 2023
* chore: adapt to leanprover/lean4#2621

* bump to v4.2.0-rc2
kim-em added a commit to leanprover-community/aesop that referenced this pull request Oct 16, 2023
* fix: use replay from leanprover/lean4#2617

* chore: adapt to leanprover/lean4#2621

* bump to v4.2.0-rc2
thorimur pushed a commit to thorimur/batteries that referenced this pull request Oct 23, 2023
thorimur pushed a commit to thorimur/batteries that referenced this pull request Oct 23, 2023
thorimur pushed a commit to thorimur/batteries that referenced this pull request Oct 24, 2023
thorimur pushed a commit to thorimur/batteries that referenced this pull request Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review Waiting for someone to review the PR builds-mathlib CI has verified that Mathlib builds against this PR 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.

Incorrect simp tracing for local right-to-left simp theorems ([<- x])

2 participants