Skip to content

Conversation

@datokrat
Copy link
Contributor

@datokrat datokrat commented Jun 13, 2025

This PR introduces a ForIn' instance and a size function for iterators in a minimal fashion. The ForIn' instance is not marked as an instance because it is unclear which Membership relation is sufficiently useful. The ForIn' instance existing as a def and inducing the ForIn instance, it becomes possible to provide more specialized ForIn' instances, with nice Membership relations, for various types of iterators. The size function has no lemmas yet.

@datokrat datokrat added the changelog-library Library label Jun 13, 2025
@datokrat datokrat changed the base branch from master to paul/base/iterators/forinprime June 13, 2025 10:37
@datokrat datokrat force-pushed the paul/iterators/forinprime branch 2 times, most recently from ef57177 to b3e6ea4 Compare June 17, 2025 20:59
@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 Jun 17, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Jun 17, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Jun 17, 2025
@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Jun 17, 2025
@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Jun 17, 2025

Mathlib CI status (docs):

  • ✅ Mathlib branch lean-pr-testing-8768 has successfully built against this PR. (2025-06-17 22:25:02) View Log
  • ✅ Mathlib branch lean-pr-testing-8768 has successfully built against this PR. (2025-06-18 09:11:15) View Log
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase 86eded35db40fadcd718eed738e0860b942841b1 --onto e83b7681407043c75f5167c2e4da13e25078a0c0. You can force Mathlib CI using the force-mathlib-ci label. (2025-06-18 12:08:29)
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase aea8e11d4ba585f74bbe5cd569b2f007751b451c --onto e83b7681407043c75f5167c2e4da13e25078a0c0. You can force Mathlib CI using the force-mathlib-ci label. (2025-06-18 19:43:03)

@datokrat datokrat marked this pull request as ready for review June 18, 2025 07:24
@datokrat datokrat requested review from TwoFX and kim-em as code owners June 18, 2025 07:24
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Jun 18, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Jun 18, 2025
@datokrat datokrat force-pushed the paul/base/iterators/forinprime branch from 5aba386 to 86eded3 Compare June 18, 2025 11:40
@datokrat datokrat force-pushed the paul/iterators/forinprime branch from 8fde7d5 to 6b335e6 Compare June 18, 2025 11:40
@datokrat datokrat force-pushed the paul/iterators/forinprime branch from 6b335e6 to ab2d854 Compare June 18, 2025 19:19
@datokrat datokrat changed the base branch from paul/base/iterators/forinprime to master June 18, 2025 19:20
@datokrat datokrat added this pull request to the merge queue Jun 18, 2025
Merged via the queue into master with commit 1b5a9be Jun 18, 2025
17 checks passed
wkrozowski pushed a commit to wkrozowski/lean4 that referenced this pull request Jun 24, 2025
This PR introduces a `ForIn'` instance and a `size` function for
iterators in a minimal fashion. The `ForIn'` instance is not marked as
an instance because it is unclear which `Membership` relation is
sufficiently useful. The `ForIn'` instance existing as a `def` and
inducing the `ForIn` instance, it becomes possible to provide more
specialized `ForIn'` instances, with nice `Membership` relations, for
various types of iterators. The `size` function has no lemmas yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-mathlib CI has verified that Mathlib builds against this PR changelog-library Library 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.

4 participants