-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
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
CompatHelper: bump compat for Zygote to 0.7 for package bench, (keep existing compat) #156
CompatHelper: bump compat for Zygote to 0.7 for package bench, (keep existing compat) #156
Conversation
c32c543
to
3f2e1cd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Benchmark Results
Benchmark suite | Current: c55528e | Previous: acf55fc | Ratio |
---|---|---|---|
normal/RepGradELBO + STL/meanfield/Zygote |
13802519072 ns |
15198724796 ns |
0.91 |
normal/RepGradELBO + STL/meanfield/ForwardDiff |
3356403413.5 ns |
3240455962.5 ns |
1.04 |
normal/RepGradELBO + STL/meanfield/ReverseDiff |
3021926516 ns |
3069781398 ns |
0.98 |
normal/RepGradELBO + STL/fullrank/Zygote |
13681473373 ns |
14849629285 ns |
0.92 |
normal/RepGradELBO + STL/fullrank/ForwardDiff |
3689636504.5 ns |
3567893647.5 ns |
1.03 |
normal/RepGradELBO + STL/fullrank/ReverseDiff |
5663417464 ns |
5663062388 ns |
1.00 |
normal/RepGradELBO/meanfield/Zygote |
6116693013 ns |
7039209760 ns |
0.87 |
normal/RepGradELBO/meanfield/ForwardDiff |
2466536350 ns |
2432843392 ns |
1.01 |
normal/RepGradELBO/meanfield/ReverseDiff |
1321012473 ns |
1324250921 ns |
1.00 |
normal/RepGradELBO/fullrank/Zygote |
6203991729 ns |
7055155643 ns |
0.88 |
normal/RepGradELBO/fullrank/ForwardDiff |
2629277548 ns |
2633336787 ns |
1.00 |
normal/RepGradELBO/fullrank/ReverseDiff |
2459356113 ns |
2446923916 ns |
1.01 |
normal + bijector/RepGradELBO + STL/meanfield/Zygote |
22084065176 ns |
23812917423 ns |
0.93 |
normal + bijector/RepGradELBO + STL/meanfield/ForwardDiff |
10546466980 ns |
9910518313 ns |
1.06 |
normal + bijector/RepGradELBO + STL/meanfield/ReverseDiff |
4530973116.5 ns |
4461054562 ns |
1.02 |
normal + bijector/RepGradELBO + STL/fullrank/Zygote |
22345253347 ns |
24066961876 ns |
0.93 |
normal + bijector/RepGradELBO + STL/fullrank/ForwardDiff |
10945544739 ns |
10881089019 ns |
1.01 |
normal + bijector/RepGradELBO + STL/fullrank/ReverseDiff |
7859941162 ns |
7750916721 ns |
1.01 |
normal + bijector/RepGradELBO/meanfield/Zygote |
13813477870 ns |
15331386356 ns |
0.90 |
normal + bijector/RepGradELBO/meanfield/ForwardDiff |
9270416271 ns |
9388134243 ns |
0.99 |
normal + bijector/RepGradELBO/meanfield/ReverseDiff |
2587262809 ns |
2544179631 ns |
1.02 |
normal + bijector/RepGradELBO/fullrank/Zygote |
14000051621 ns |
15668964233 ns |
0.89 |
normal + bijector/RepGradELBO/fullrank/ForwardDiff |
9875600981 ns |
9871406635 ns |
1.00 |
normal + bijector/RepGradELBO/fullrank/ReverseDiff |
4083240829 ns |
4050689049.5 ns |
1.01 |
This comment was automatically generated by workflow using github-action-benchmark.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #156 +/- ##
==========================================
+ Coverage 91.76% 93.08% +1.32%
==========================================
Files 13 13
Lines 352 347 -5
==========================================
Hits 323 323
+ Misses 29 24 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…xisting compat) (#157) Co-authored-by: CompatHelper Julia <[email protected]>
This pull request changes the compat entry for the
Zygote
package from0.6
to0.6, 0.7
for package bench.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.