Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merged by Bors] - chore: classify porting notes about additional necessary beta reduction #12130

Closed
wants to merge 8 commits into from

Conversation

grunweg
Copy link
Collaborator

@grunweg grunweg commented Apr 14, 2024

This subsumes some of the notes in #10752 and #10971.
I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful; this is easy to revert if needed.


Commits are mostly independent.


Open in Gitpod

I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful, hence have split this into a separate commit.
Some of these notes were not true, and the best way to make them not regress
is to just do beta reduction.

This includes all remaining instances of 10971.
@grunweg grunweg changed the title Mr porting notes beta reduce chore: classify porting notes about additional necessary beta reduction Apr 14, 2024
@grunweg grunweg added awaiting-review The author would like community review of the PR porting-notes Mathlib3 to Mathlib4 porting notes. labels Apr 14, 2024
@semorrison
Copy link
Contributor

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Apr 15, 2024
mathlib-bors bot pushed a commit that referenced this pull request Apr 15, 2024
…ion (#12130)

This subsumes some of the notes in #10752 and #10971.
I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful; this is easy to revert if needed.
@mathlib-bors
Copy link
Contributor

mathlib-bors bot commented Apr 15, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title chore: classify porting notes about additional necessary beta reduction [Merged by Bors] - chore: classify porting notes about additional necessary beta reduction Apr 15, 2024
@mathlib-bors mathlib-bors bot closed this Apr 15, 2024
@mathlib-bors mathlib-bors bot deleted the MR-porting-notes-beta-reduce branch April 15, 2024 07:51
Louddy pushed a commit that referenced this pull request Apr 15, 2024
…ion (#12130)

This subsumes some of the notes in #10752 and #10971.
I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful; this is easy to revert if needed.
atarnoam pushed a commit that referenced this pull request Apr 16, 2024
…ion (#12130)

This subsumes some of the notes in #10752 and #10971.
I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful; this is easy to revert if needed.
uniwuni pushed a commit that referenced this pull request Apr 19, 2024
…ion (#12130)

This subsumes some of the notes in #10752 and #10971.
I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful; this is easy to revert if needed.
callesonne pushed a commit that referenced this pull request Apr 22, 2024
…ion (#12130)

This subsumes some of the notes in #10752 and #10971.
I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful; this is easy to revert if needed.
Jun2M pushed a commit that referenced this pull request Apr 24, 2024
…ion (#12130)

This subsumes some of the notes in #10752 and #10971.
I'm on the fence as to whether replacing the dsimp only by beta_reduce is useful; this is easy to revert if needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
porting-notes Mathlib3 to Mathlib4 porting notes. ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants