8000 feat: `BaseIO.joinTask` by tydeu · Pull Request #8070 · leanprover/lean4 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat: BaseIO.joinTask #8070

New issue 8000

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

feat: BaseIO.joinTask #8070

wants to merge 3 commits into from

Conversation

tydeu
Copy link
Member
@tydeu tydeu commented Apr 23, 2025

This PR adds BaseIO.joinTask act prio, an optimized version of (<- act.asTask prio).bind id.

@tydeu tydeu added the changelog-library Library label Apr 23, 2025
@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 Apr 24, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Apr 24, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Apr 24, 2025
@leanprover-community-bot leanprover-community-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Apr 24, 2025
@leanprover-community-bot
Copy link
Collaborator
leanprover-community-bot commented Apr 24, 2025

Mathlib CI status (docs):

@tydeu
Copy link
Member Author
tydeu commented Apr 24, 2025

!bench

@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit 6e1436e.
There were significant changes against commit 02f7a1d:

  Benchmark                            Metric                    Change
  ================================================================================
- Init.Data.List.Sublist re-elab -j4   maxrss                      9.2%  (149.1 σ)
+ big_do                               task-clock                 -4.1%  (-16.5 σ)
+ big_do                               wall-clock                 -4.1%  (-11.3 σ)
+ bv_decide_large_aig.lean             branch-misses              -1.5%  (-24.5 σ)
- channel.lean                         bounded0_spsc              15.2%   (13.6 σ)
+ lake build clean                     task-clock                 -1.6%  (-16.1 σ)
+ lake build clean                     wall-clock                 -2.2%  (-16.4 σ)
+ omega_stress.lean async              branch-misses              -2.0%  (-11.0 σ)
+ stdlib                               attribute application      -2.4%  (-12.9 σ)
+ stdlib                               blocked                    -3.1%  (-60.1 σ)
+ stdlib                               blocked (unaccounted)      -2.8%  (-12.1 σ)
+ stdlib                               instantiate metavars       -4.3%  (-10.8 σ)
+ stdlib                               process pre-definitions    -3.8%  (-77.7 σ)
+ stdlib                               tactic execution           -2.7%  (-34.0 σ)
+ stdlib                               task-clock                 -2.9% (-573.4 σ)
+ stdlib                               type checking              -3.6%  (-19.7 σ)

@tydeu
Copy link
Member Author
tydeu commented Apr 25, 2025

!bench

@leanprover-bot

This comment was marked as duplicate.

@tydeu
Copy link
Member Author
tydeu commented Apr 25, 2025

!bench

leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Apr 25, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Apr 25, 2025
@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit 94f6ae6.
There were significant changes against commit 02f7a1d:

  Benchmark                            Metric       Change
  ==================================================================
- Init.Data.List.Sublist re-elab -j4   maxrss         9.4%  (20.5 σ)
+ bv_decide_mul                        maxrss        -2.0% (-59.9 σ)
- lake build no-op                     wall-clock    13.8%  (13.5 σ)
+ liasolver                            task-clock    -1.4% (-31.2 σ)
+ liasolver                            wall-clock    -1.4% (-35.0 σ)
+ qsort                                task-clock    -4.2% (-19.5 σ)
+ qsort                                wall-clock    -4.2% (-19.7 σ)
- workspaceSymbols                     task-clock     2.7%  (16.0 σ)
- workspaceSymbols                     wall-clock     2.7%  (16.0 σ)

@tydeu
Copy link
Member Author
tydeu commented Jun 1, 2025

!bench

leanprover-community-mathlib4-bot added a commit to leanprover-community/batteries that referenced this pull request Jun 1, 2025
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Jun 1, 2025
@leanprover-bot
Copy link
Collaborator

Here are the benchmark results for commit 39aaa68.
There were significant changes against commit 72141b0:

  Benchmark      Metric                    Change
  =========================================================
+ channel.lean   maxrss                     -2.6% (-23.7 σ)
+ liasolver      maxrss                     -2.6% (-70.6 σ)
+ nat_repr       maxrss                     -2.5% (-92.0 σ)
+ qsort          maxrss                     -2.6% (-70.6 σ)
+ stdlib         process pre-definitions    -1.5% (-20.6 σ)

@tydeu tydeu marked this pull request as ready for review June 1, 2025 21:50
@tydeu tydeu requested a review from Kha June 1, 2025 21:50
@Kha Kha requested a review from hargoniX June 2, 2025 08:14
@Kha
Copy link
Member
Kha commented Jun 2, 2025

What is the greater context here? Is it measurable?

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
0