Skip to content

CompatHelper: bump compat for Revise in [extras] to 3, (keep existing compat)#920

Open
github-actions[bot] wants to merge 7 commits intomainfrom
compathelper/new_version/2026-01-04-00-07-12-054-00396649051
Open

CompatHelper: bump compat for Revise in [extras] to 3, (keep existing compat)#920
github-actions[bot] wants to merge 7 commits intomainfrom
compathelper/new_version/2026-01-04-00-07-12-054-00396649051

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Jan 4, 2026

Note (@penelopeysm): Revise 3.13 breaks allocation tests, do not merge until that is understood or resolved

This pull request changes the compat entry for the Revise package from 3.0 - 3.12 to 3.0 - 3.12, 3.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

CI Summary — GitHub Actions

Documentation Preview

Mooncake.jl documentation for PR #920 is available at:
https://chalk-lab.github.io/Mooncake.jl/previews/PR920/

Performance

Performance Ratio:
Ratio of time to compute gradient and time to compute function.
Warning: results are very approximate! See here for more context.

┌────────────────────────────┬──────────┬──────────┬─────────────┬─────────┬─────────────┬────────┐
│                      Label │   Primal │ Mooncake │ MooncakeFwd │  Zygote │ ReverseDiff │ Enzyme │
│                     String │   String │   String │      String │  String │      String │ String │
├────────────────────────────┼──────────┼──────────┼─────────────┼─────────┼─────────────┼────────┤
│                   sum_1000 │ 170.0 ns │     1.71 │        1.71 │   0.706 │        3.54 │    6.6 │
│                  _sum_1000 │ 952.0 ns │     6.82 │        1.01 │  1700.0 │        33.6 │   1.09 │
│               sum_sin_1000 │  7.31 μs │     3.33 │        1.45 │    1.58 │        11.0 │   1.75 │
│              _sum_sin_1000 │  5.61 μs │     4.15 │        2.11 │   269.0 │        12.8 │   2.33 │
│                   kron_sum │ 220.0 μs │     12.3 │        3.46 │    23.2 │       311.0 │   22.4 │
│              kron_view_sum │ 299.0 μs │     10.4 │        4.81 │    24.8 │       308.0 │   12.8 │
│      naive_map_sin_cos_exp │   2.6 μs │     2.73 │        1.37 │ missing │        6.22 │   1.96 │
│            map_sin_cos_exp │  2.59 μs │     3.02 │        1.39 │    1.97 │        5.22 │   2.45 │
│      broadcast_sin_cos_exp │  2.72 μs │     2.72 │        1.39 │    3.89 │        1.22 │   1.86 │
│                 simple_mlp │ 407.0 μs │     4.47 │        2.64 │    1.39 │        7.47 │   2.73 │
│                     gp_lml │ 220.0 μs │     10.7 │        2.58 │    5.14 │     missing │   7.03 │
│ turing_broadcast_benchmark │  2.17 ms │     4.55 │        2.83 │ missing │        22.9 │   1.86 │
│         large_single_block │ 401.0 ns │     5.75 │        1.97 │  4370.0 │        30.7 │   2.15 │
└────────────────────────────┴──────────┴──────────┴─────────────┴─────────┴─────────────┴────────┘

@willtebbutt willtebbutt force-pushed the compathelper/new_version/2026-01-04-00-07-12-054-00396649051 branch from 25242f1 to 044f301 Compare January 4, 2026 00:07
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Jan 4, 2026

Mooncake.jl documentation for PR #920 is available at:
https://chalk-lab.github.io/Mooncake.jl/previews/PR920/

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Jan 4, 2026

Performance Ratio:
Ratio of time to compute gradient and time to compute function.
Warning: results are very approximate! See here for more context.

┌────────────────────────────┬──────────┬──────────┬─────────────┬─────────┬─────────────┬────────┐
│                      Label │   Primal │ Mooncake │ MooncakeFwd │  Zygote │ ReverseDiff │ Enzyme │
│                     String │   String │   String │      String │  String │      String │ String │
├────────────────────────────┼──────────┼──────────┼─────────────┼─────────┼─────────────┼────────┤
│                   sum_1000 │ 110.0 ns │     1.91 │        1.92 │    1.09 │        5.28 │   10.2 │
│                  _sum_1000 │ 952.0 ns │     6.94 │        1.01 │  1590.0 │        39.3 │   1.08 │
│               sum_sin_1000 │  6.55 μs │     2.74 │        1.38 │    1.81 │        11.3 │   2.15 │
│              _sum_sin_1000 │   5.3 μs │     3.07 │        2.19 │   287.0 │        13.7 │   2.48 │
│                   kron_sum │ 396.0 μs │     7.44 │        2.92 │    4.48 │       173.0 │   10.9 │
│              kron_view_sum │ 401.0 μs │     8.48 │        4.22 │    18.8 │       176.0 │   9.81 │
│      naive_map_sin_cos_exp │  2.19 μs │     2.45 │        1.43 │ missing │        7.27 │   2.32 │
│            map_sin_cos_exp │  2.24 μs │     2.73 │        1.45 │    1.66 │         5.9 │   2.85 │
│      broadcast_sin_cos_exp │  2.33 μs │      2.4 │        1.42 │    3.89 │        1.51 │   2.19 │
│                 simple_mlp │ 449.0 μs │     4.48 │        2.95 │    1.66 │        8.24 │   3.26 │
│                     gp_lml │ 592.0 μs │     4.23 │        1.51 │    2.17 │     missing │   3.03 │
│ turing_broadcast_benchmark │  1.98 ms │      4.1 │        3.09 │ missing │        24.0 │   2.02 │
│         large_single_block │ 391.0 ns │     4.46 │         2.0 │  4650.0 │        36.2 │    2.2 │
└────────────────────────────┴──────────┴──────────┴─────────────┴─────────┴─────────────┴────────┘

@penelopeysm penelopeysm marked this pull request as draft January 4, 2026 16:08
@yebai yebai force-pushed the main branch 3 times, most recently from dfec3c4 to 7b2f7ba Compare January 24, 2026 21:38
…-054-00396649051

Signed-off-by: Hong Ge <3279477+yebai@users.noreply.github.com>
@yebai
Copy link
Copy Markdown
Member

yebai commented Mar 24, 2026

Let's see whether #1093 and #1094 fixed the inaccurate allocation issues.

Signed-off-by: Hong Ge <3279477+yebai@users.noreply.github.com>
@yebai yebai marked this pull request as ready for review March 24, 2026 17:39
yebai and others added 3 commits March 24, 2026 18:13
Signed-off-by: Hong Ge <3279477+yebai@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant