Skip to content

chore: adaptations for nightly-2024-02-26#10995

Merged
kmill merged 52 commits intobump/v4.7.0from
bump/nightly-2024-02-26
Feb 28, 2024
Merged

chore: adaptations for nightly-2024-02-26#10995
kmill merged 52 commits intobump/v4.7.0from
bump/nightly-2024-02-26

Conversation

@kim-em
Copy link
Copy Markdown
Contributor

@kim-em kim-em commented Feb 27, 2024

No description provided.

@kim-em kim-em changed the base branch from bump/v4.7.0 to master February 27, 2024 04:22
@kim-em kim-em changed the base branch from master to bump/v4.7.0 February 27, 2024 04:23
kim-em added a commit that referenced this pull request Feb 27, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 27, 2024
Co-authored-by: Patrick Massot <patrickmassot@free.fr>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
ext n; simp only [coeff_smul, coeff_scaleRoots, coeff_add, smul_eq_mul,
mul_comm (r ^ _), mul_assoc, ← pow_add, ← h, ← add_mul, add_comm (_ - n)]
mul_comm (r ^ _), ← pow_add, ← h, ← add_mul, add_comm (_ - n)]
rw [mul_assoc, ← pow_add]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why do these not fire in simp now?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

They do fire, but mul_assoc induces a max rec depth error.

I've added an "adaptation note" explaining this, but am inclined to ship.

Copy link
Copy Markdown
Contributor

@mattrobball mattrobball left a comment

Choose a reason for hiding this comment

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

Good with me

@kmill kmill merged commit b700984 into bump/v4.7.0 Feb 28, 2024
@mathlib-bors mathlib-bors bot deleted the bump/nightly-2024-02-26 branch February 28, 2024 05:08
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
Co-authored-by: Patrick Massot <patrickmassot@free.fr>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
kbuzzard pushed a commit that referenced this pull request Mar 12, 2024
Co-authored-by: Patrick Massot <patrickmassot@free.fr>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
Co-authored-by: Patrick Massot <patrickmassot@free.fr>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
xgenereux pushed a commit that referenced this pull request Apr 15, 2024
Co-authored-by: Patrick Massot <patrickmassot@free.fr>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
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.

5 participants