Skip to content

improve performance of hessians with static arrays - #751

Merged
devmotion merged 5 commits into
JuliaDiff:masterfrom
longemen3000:static-hessian
Sep 9, 2025
Merged

improve performance of hessians with static arrays#751
devmotion merged 5 commits into
JuliaDiff:masterfrom
longemen3000:static-hessian

Conversation

@longemen3000

Copy link
Copy Markdown
Contributor

by adding the suggestion done in #720 .

Closes#720

@DNF2

DNF2 commented Sep 9, 2025

Copy link
Copy Markdown

Is there any chance of this being merged? It has a significant impact on my library.

@devmotion

Copy link
Copy Markdown
Member

All tests are failing currently.

@codecov

codecovBot commented Sep 9, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.54%. Comparing base (42e0aa6) to head (a6306bf).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #751 +/- ##
==========================================
- Coverage 89.82% 89.54% -0.29% 
==========================================
Files 11 11 Lines 1032 1033 +1 ==========================================
- Hits 927 925 -2 - Misses 105 108 +3 

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

@devmotion
devmotion merged commit d0d6a0a into JuliaDiff:masterSep 9, 2025
11 of 12 checks passed
@nsajkonsajko mentioned this pull request Sep 9, 2025
Merged
devmotion pushed a commit that referenced this pull request Sep 9, 2025
Release PRs #771 and #751.
Specifically I want a release because of the former, to cut down on
sysimage invalidation when loading ForwardDiff.
@longemen3000
longemen3000 deleted the static-hessian branch September 10, 2025 03:38
Sign up for freeto 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.

Performance issue/allocations in DiffResults-hessian! with StaticArrays.

3 participants

@longemen3000@DNF2@devmotion