Skip to content

Conversation

@datokrat
Copy link
Contributor

@datokrat datokrat commented Dec 16, 2025

This PR removes the IteratorCollect type class and hereby simplifies the iterator API. Its limited advantages did not justify the complexity cost.

@datokrat datokrat added the changelog-library Library label Dec 16, 2025
@datokrat
Copy link
Contributor Author

!radar

@leanprover-radar
Copy link

leanprover-radar commented Dec 16, 2025

Benchmark results for 1783806 against e261790 are in! @datokrat

Minor changes (3)
  • iterators (elab)//instructions: -20.8M (-0.5%)
  • size/init/.olean.private//bytes: -521kiB (-0.2%)
  • size/init/.olean//bytes: -224kiB (-0.2%)

@datokrat datokrat force-pushed the paul/iterators/remove-iteratorcollect branch from 1783806 to 545ce11 Compare December 17, 2025 07:17
@datokrat datokrat changed the base branch from master to paul/base/iterators/remove-iteratorcollect December 17, 2025 07:17
@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 Dec 17, 2025
@leanprover-community-bot
Copy link
Collaborator

leanprover-community-bot commented Dec 17, 2025

Mathlib CI status (docs):

  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase e2617903f81f2f9ffc6c0b7a06cc91678c546305 --onto 95a7c769d8d1565fdafd32553bb44d7d53edf5c3. You can force Mathlib CI using the force-mathlib-ci label. (2025-12-17 08:15:24)
  • ❗ Batteries/Mathlib CI will not be attempted unless your PR branches off the nightly-with-mathlib branch. Try git rebase f21f8d96f97989d3a34148329a158e94b220314b --onto 0708024c4649f11da1276d8b6d4345972a9dc57c. You can force Mathlib CI using the force-mathlib-ci label. (2025-12-17 23:12:24)

@leanprover-bot
Copy link
Collaborator

leanprover-bot commented Dec 17, 2025

Reference manual CI status:

  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase e2617903f81f2f9ffc6c0b7a06cc91678c546305 --onto 95a7c769d8d1565fdafd32553bb44d7d53edf5c3. You can force reference manual CI using the force-manual-ci label. (2025-12-17 08:15:26)
  • ❗ Reference manual CI will not be attempted unless your PR branches off the nightly-with-manual branch. Try git rebase f21f8d96f97989d3a34148329a158e94b220314b --onto 0708024c4649f11da1276d8b6d4345972a9dc57c. You can force reference manual CI using the force-manual-ci label. (2025-12-17 23:12:25)

@datokrat datokrat changed the base branch from paul/base/iterators/remove-iteratorcollect to master December 17, 2025 22:16
@datokrat datokrat force-pushed the paul/iterators/remove-iteratorcollect branch from 545ce11 to 8af8fd5 Compare December 17, 2025 22:16
@github-actions github-actions bot added the changes-stage0 Contains stage0 changes, merge manually using rebase label Dec 17, 2025
@github-actions github-actions bot removed the changes-stage0 Contains stage0 changes, merge manually using rebase label Dec 17, 2025
@datokrat datokrat marked this pull request as ready for review December 17, 2025 23:01
@datokrat datokrat added this pull request to the merge queue Dec 17, 2025
Merged via the queue into master with commit 5ef0207 Dec 17, 2025
20 checks passed
robsimmons added a commit to leanprover/reference-manual that referenced this pull request Dec 20, 2025
@robsimmons
Copy link
Contributor

@datokrat can you spot check leanprover/reference-manual#715 when you get a chance in light of this?

@datokrat
Copy link
Contributor Author

@robsimmons Looking good! Sorry for not leaving more breadcrumbs for how to solve the errors in the manual.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

6 participants