Skip to content

[Merged by Bors] - chore(*): go through adaptation notes for lean4#6024#27348

Closed
Vierkantor wants to merge 1 commit intoleanprover-community:masterfrom
Vierkantor:triage-adaptation_note-6024
Closed

[Merged by Bors] - chore(*): go through adaptation notes for lean4#6024#27348
Vierkantor wants to merge 1 commit intoleanprover-community:masterfrom
Vierkantor:triage-adaptation_note-6024

Conversation

@Vierkantor
Copy link
Copy Markdown
Contributor

@Vierkantor Vierkantor commented Jul 22, 2025

We have some adaptation notes referring to lean4#6024, which changed elaboration subtly; some tricky unifications no longer get solved. I checked every reference to make sure that these still apply. We don't expect that this change gets reverted, so we can turn the remaining adaptation notes into regular comments.


Open in Gitpod

We have some adaptation notes referring to lean4#6024, which changed elaboration subtly; some tricky unifications no longer get solved. I checked every reference to make sure that these still apply. We don't expect that this change gets reverted, so we can turn the remaining adaptation notes into regular comments.
@Vierkantor Vierkantor added the tech debt Tracking cross-cutting technical debt, see e.g. the "Technical debt counters" stream on zulip label Jul 22, 2025
@github-actions
Copy link
Copy Markdown

PR summary ffe8eee4d3

Import changes for modified files

No significant changes to the import graph

Import changes for all files
Files Import difference

Declarations diff

No declarations were harmed in the making of this PR! 🐙

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.


Decrease in tech debt: (relative, absolute) = (33.00, 0.39)
Current number Change Type
84 -33 adaptation notes

Current commit 19b2201b00
Reference commit ffe8eee4d3

You can run this locally as

./scripts/technical-debt-metrics.sh pr_summary
  • The relative value is the weighted sum of the differences with weight given by the inverse of the current value of the statistic.
  • The absolute value is the relative value divided by the total sum of the inverses of the current values (i.e. the weighted average of the differences).

@bryangingechen
Copy link
Copy Markdown
Contributor

Thanks!
bors r+

@ghost ghost added the ready-to-merge This PR has been sent to bors. label Jul 22, 2025
mathlib-bors bot pushed a commit that referenced this pull request Jul 22, 2025
We have some adaptation notes referring to [lean4#6024](leanprover/lean4#6024), which changed elaboration subtly; some tricky unifications no longer get solved. I checked every reference to make sure that these still apply. We don't expect that this change gets reverted, so we can turn the remaining adaptation notes into regular comments.
@mathlib-bors
Copy link
Copy Markdown
Contributor

mathlib-bors bot commented Jul 22, 2025

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore(*): go through adaptation notes for lean4#6024 [Merged by Bors] - chore(*): go through adaptation notes for lean4#6024 Jul 22, 2025
@mathlib-bors mathlib-bors bot closed this Jul 22, 2025
callesonne pushed a commit to callesonne/mathlib4 that referenced this pull request Jul 24, 2025
…unity#27348)

We have some adaptation notes referring to [lean4#6024](leanprover/lean4#6024), which changed elaboration subtly; some tricky unifications no longer get solved. I checked every reference to make sure that these still apply. We don't expect that this change gets reverted, so we can turn the remaining adaptation notes into regular comments.
hrmacbeth pushed a commit to szqzs/mathlib4 that referenced this pull request Jul 28, 2025
…unity#27348)

We have some adaptation notes referring to [lean4#6024](leanprover/lean4#6024), which changed elaboration subtly; some tricky unifications no longer get solved. I checked every reference to make sure that these still apply. We don't expect that this change gets reverted, so we can turn the remaining adaptation notes into regular comments.
Equilibris pushed a commit to Equilibris/mathlib4 that referenced this pull request Aug 7, 2025
…unity#27348)

We have some adaptation notes referring to [lean4#6024](leanprover/lean4#6024), which changed elaboration subtly; some tricky unifications no longer get solved. I checked every reference to make sure that these still apply. We don't expect that this change gets reverted, so we can turn the remaining adaptation notes into regular comments.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge This PR has been sent to bors. tech debt Tracking cross-cutting technical debt, see e.g. the "Technical debt counters" stream on zulip

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants