Skip to content
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

Fix DataDrivenDiffEq.jl downstream test #559

Merged

Conversation

contradict
Copy link
Contributor

"Standard" group no longer exists in DataDrivenDiffEq.jl, "Core" looks like a good replacement.

"Standard" group no longer exists in DataDrivenDiffEq.jl, "Core" looks
like a good replacement.
@contradict contradict mentioned this pull request Oct 13, 2023
@github-actions
Copy link
Contributor

Benchmark Results

master f43ec8c... t[master]/t[f43ec8c...]
overhead/acrule/a+2 1.06 ± 0.26 μs 1.08 ± 0.26 μs 0.982
overhead/acrule/a+2+b 1.05 ± 0.24 μs 1.07 ± 0.26 μs 0.985
overhead/acrule/a+b 0.354 ± 0.0079 μs 0.356 ± 0.0066 μs 0.995
overhead/acrule/noop:Int 26.9 ± 0.5 ns 26.7 ± 0.4 ns 1.01
overhead/acrule/noop:Sym 0.0457 ± 0.008 μs 0.0455 ± 0.0076 μs 1
overhead/rule/noop:Int 0.0443 ± 0.0016 μs 0.045 ± 0.0019 μs 0.985
overhead/rule/noop:Sym 0.0606 ± 0.0018 μs 0.0571 ± 0.0018 μs 1.06
overhead/rule/noop:Term 0.0581 ± 0.0019 μs 0.0577 ± 0.0018 μs 1.01
overhead/ruleset/noop:Int 0.168 ± 0.0018 μs 0.165 ± 0.0019 μs 1.01
overhead/ruleset/noop:Sym 0.187 ± 0.0042 μs 0.194 ± 0.0054 μs 0.96
overhead/ruleset/noop:Term 4.43 ± 0.31 μs 4.54 ± 0.31 μs 0.975
overhead/simplify/noop:Int 0.228 ± 0.0077 μs 0.232 ± 0.018 μs 0.982
overhead/simplify/noop:Sym 0.248 ± 0.0051 μs 0.243 ± 0.0052 μs 1.02
overhead/simplify/noop:Term 0.0447 ± 0.0036 ms 0.0456 ± 0.0037 ms 0.98
overhead/simplify/randterm (+, *):serial 0.159 ± 0.0015 s 0.163 ± 0.0014 s 0.981
overhead/simplify/randterm (+, *):thread 0.111 ± 0.015 s 0.111 ± 0.016 s 0.994
overhead/simplify/randterm (/, *):serial 0.278 ± 0.0082 ms 0.282 ± 0.0086 ms 0.986
overhead/simplify/randterm (/, *):thread 0.316 ± 0.0096 ms 0.323 ± 0.0097 ms 0.981
overhead/substitute/a 0.0748 ± 0.0015 ms 0.0741 ± 0.0014 ms 1.01
overhead/substitute/a,b 0.0651 ± 0.0015 ms 0.0643 ± 0.0015 ms 1.01
overhead/substitute/a,b,c 21.4 ± 0.6 μs 22.3 ± 0.7 μs 0.96
polyform/easy_iszero 0.0403 ± 0.0031 ms 0.0395 ± 0.003 ms 1.02
polyform/isone 3 ± 0.1 ns 3.1 ± 0.1 ns 0.968
polyform/iszero 2.46 ± 0.03 ms 2.44 ± 0.029 ms 1.01
polyform/simplify_fractions 3.3 ± 0.036 ms 3.29 ± 0.032 ms 1.01
time_to_load 5.46 ± 0.11 s 5.36 ± 0.035 s 1.02

Benchmark Plots

A plot of the benchmark results have been uploaded as an artifact to the workflow run for this PR.
Go to "Actions"->"Benchmark a pull request"->[the most recent run]->"Artifacts" (at the bottom).

Copy link
Member

@bowenszhu bowenszhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bowenszhu bowenszhu merged commit d2f87a0 into JuliaSymbolics:master May 8, 2024
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.

2 participants