Skip to content

Conversation

@devmotion
Copy link
Contributor

No description provided.

@pkofod
Copy link
Member

pkofod commented Nov 2, 2025

the tests that fail are quite sensitive / unstable

@devmotion
Copy link
Contributor Author

The failing macOS test uses the native aarch64 binaries in this PR but currently on master runs with Rosetta emulation.

@pkofod
Copy link
Member

pkofod commented Nov 2, 2025

I can make a commit to adjust the tests accordingly

- "min"
- "lts"
- "1"
- "pre"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's a bit questionable whether to always run tests on "pre" - when there's no new prerelease available (e.g. for 1.13), then tests will be identical to "1".

Copy link
Member

Choose a reason for hiding this comment

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

Feel free to remove it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I removed it in a4c4adc

@devmotion
Copy link
Contributor Author

I can make a commit to adjust the tests accordingly

That would be great, I'm not familiar with that part of Optim yet.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2025

Benchmark Results (Julia vlts)

Time benchmarks
master a4c4adc... master / a4c4adc...
multivariate/solvers/first_order/AdaMax 0.542 ± 0.0095 ms 0.543 ± 0.0095 ms 0.998 ± 0.025
multivariate/solvers/first_order/Adam 0.542 ± 0.0095 ms 0.542 ± 0.0094 ms 1 ± 0.025
multivariate/solvers/first_order/BFGS 0.264 ± 0.0085 ms 0.263 ± 0.0083 ms 1 ± 0.045
multivariate/solvers/first_order/ConjugateGradient 0.174 ± 0.0029 ms 0.174 ± 0.0029 ms 0.999 ± 0.024
multivariate/solvers/first_order/GradientDescent 1.55 ± 0.012 ms 1.55 ± 0.012 ms 1 ± 0.011
multivariate/solvers/first_order/LBFGS 0.233 ± 0.0081 ms 0.232 ± 0.0077 ms 1 ± 0.048
multivariate/solvers/first_order/MomentumGradientDescent 2.17 ± 0.016 ms 2.17 ± 0.016 ms 1 ± 0.01
multivariate/solvers/first_order/NGMRES 0.431 ± 0.011 ms 0.431 ± 0.011 ms 0.998 ± 0.036
time_to_load 0.423 ± 0.0044 s 0.435 ± 0.0039 s 0.973 ± 0.013
Memory benchmarks
master a4c4adc... master / a4c4adc...
multivariate/solvers/first_order/AdaMax 0.34 k allocs: 7.16 kB 0.34 k allocs: 7.16 kB 1
multivariate/solvers/first_order/Adam 0.34 k allocs: 7.16 kB 0.34 k allocs: 7.16 kB 1
multivariate/solvers/first_order/BFGS 0.336 k allocs: 15 kB 0.336 k allocs: 15 kB 1
multivariate/solvers/first_order/ConjugateGradient 0.332 k allocs: 13.5 kB 0.332 k allocs: 13.5 kB 1
multivariate/solvers/first_order/GradientDescent 1.89 k allocs: 0.0713 MB 1.89 k allocs: 0.0713 MB 1
multivariate/solvers/first_order/LBFGS 0.317 k allocs: 14.2 kB 0.317 k allocs: 14.2 kB 1
multivariate/solvers/first_order/MomentumGradientDescent 2.24 k allocs: 0.077 MB 2.24 k allocs: 0.077 MB 1
multivariate/solvers/first_order/NGMRES 1.51 k allocs: 0.117 MB 1.51 k allocs: 0.117 MB 1
time_to_load 0.153 k allocs: 14.5 kB 0.153 k allocs: 14.5 kB 1

A plot of the benchmark results has been uploaded as an artifact at .

@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.85%. Comparing base (c3ba21c) to head (a4c4adc).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1196      +/-   ##
==========================================
+ Coverage   87.59%   87.85%   +0.25%     
==========================================
  Files          45       45              
  Lines        3515     3515              
==========================================
+ Hits         3079     3088       +9     
+ Misses        436      427       -9     

☔ 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.

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