Skip to content

Update DynamicPPL version in Project.toml#2673

Merged
wsmoses merged 2 commits intomainfrom
wsmoses-patch-5
Oct 21, 2025
Merged

Update DynamicPPL version in Project.toml#2673
wsmoses merged 2 commits intomainfrom
wsmoses-patch-5

Conversation

@wsmoses
Copy link
Member

@wsmoses wsmoses commented Oct 17, 2025

No description provided.

@github-actions
Copy link
Contributor

Benchmark Results

main 905062d... main / 905062d...
basics/make_zero/namedtuple 0.0529 ± 0.0026 μs 0.0528 ± 0.0025 μs 1 ± 0.069
basics/make_zero/struct 0.25 ± 0.0061 μs 0.253 ± 0.0057 μs 0.988 ± 0.033
basics/overhead 4.34 ± 0.01 ns 4.34 ± 0.01 ns 1 ± 0.0033
basics/remake_zero!/namedtuple 0.24 ± 0.0091 μs 0.24 ± 0.011 μs 0.998 ± 0.059
basics/remake_zero!/struct 0.231 ± 0.0078 μs 0.233 ± 0.01 μs 0.991 ± 0.055
fold_broadcast/multidim_sum_bcast/1D 10.3 ± 0.25 μs 10.3 ± 0.25 μs 1 ± 0.035
fold_broadcast/multidim_sum_bcast/2D 12.3 ± 0.3 μs 12.2 ± 0.29 μs 1.01 ± 0.034
time_to_load 1.26 ± 0.0042 s 1.26 ± 0.0092 s 1 ± 0.008

Benchmark Plots

A plot of the benchmark results has been uploaded as an artifact at https://github.com/EnzymeAD/Enzyme.jl/actions/runs/18605381941/artifacts/4304301108.

@codecov
Copy link

codecov bot commented Oct 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.14%. Comparing base (0d9ed77) to head (905062d).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2673   +/-   ##
=======================================
  Coverage   75.14%   75.14%           
=======================================
  Files          57       57           
  Lines       17958    17958           
=======================================
  Hits        13495    13495           
  Misses       4463     4463           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@giordano
Copy link
Member

The extension isn't really tested, is it?

@penelopeysm
Copy link
Contributor

penelopeysm commented Oct 21, 2025

The extension isn't really tested, is it?

No, also it's type piracy.

This compat bound is making DynamicPPL CI fail on a number of PRs. I tested with Enzyme 0.13.86 manually and the error has already been fixed in some recent patch. So the compat needs to be fixed or I'll have to start removing Enzyme from our CI benchmarks

@wsmoses
Copy link
Member Author

wsmoses commented Oct 21, 2025 via email

@wsmoses wsmoses merged commit 833309b into main Oct 21, 2025
44 of 46 checks passed
@wsmoses wsmoses deleted the wsmoses-patch-5 branch October 21, 2025 16:53
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.

3 participants