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

feat: List.merge and lemmas #579

Merged
merged 1 commit into from
Mar 6, 2024
Merged

Conversation

fgdorais
Copy link
Collaborator

@fgdorais fgdorais commented Feb 4, 2024

List.merge the merge step from merge sort (without any ordering assumptions). This will be useful to streamline and optimize ac_rfl.

Note that List.merge exists in Mathlib but the Mathlib implementation is not tail recursive and it uses a decidable relation instead of a Bool relation. A Mathlib adaptation PR is in progress: https://github.com/leanprover-community/mathlib4/tree/fgdorais-patch-list-merge

@fgdorais fgdorais added the awaiting-review This PR is ready for review; the author thinks it is ready to be merged. label Feb 4, 2024
@fgdorais
Copy link
Collaborator Author

The ac_rfl move has been canceled so there is no immediate need for this. It's still a useful addition so I'm reluctant to close but closing might be better than rotting.

@kim-em kim-em merged commit 28f4267 into leanprover-community:main Mar 6, 2024
1 check passed
Ruben-VandeVelde added a commit to leanprover-community/mathlib4 that referenced this pull request Mar 12, 2024
Std has added a different definition with the same name in
leanprover-community/batteries#579. It is easier to preemptively
rename the version in mathlib.
mathlib-bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 13, 2024
This if @fgdorais's patch for leanprover-community/batteries#579.

Co-authored-by: F. G. Dorais <[email protected]>
Co-authored-by: Scott Morrison <[email protected]>
dagurtomas pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 22, 2024
This if @fgdorais's patch for leanprover-community/batteries#579.

Co-authored-by: F. G. Dorais <[email protected]>
Co-authored-by: Scott Morrison <[email protected]>
utensil pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 26, 2024
This if @fgdorais's patch for leanprover-community/batteries#579.

Co-authored-by: F. G. Dorais <[email protected]>
Co-authored-by: Scott Morrison <[email protected]>
Louddy pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 15, 2024
This if @fgdorais's patch for leanprover-community/batteries#579.

Co-authored-by: F. G. Dorais <[email protected]>
Co-authored-by: Scott Morrison <[email protected]>
uniwuni pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 19, 2024
This if @fgdorais's patch for leanprover-community/batteries#579.

Co-authored-by: F. G. Dorais <[email protected]>
Co-authored-by: Scott Morrison <[email protected]>
callesonne pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 22, 2024
This if @fgdorais's patch for leanprover-community/batteries#579.

Co-authored-by: F. G. Dorais <[email protected]>
Co-authored-by: Scott Morrison <[email protected]>
github-merge-queue bot pushed a commit to leanprover/lean4 that referenced this pull request Sep 18, 2024
github-merge-queue bot pushed a commit to leanprover/lean4 that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review This PR is ready for review; the author thinks it is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants