Skip to content

Use FiniteDiff.finite_difference_jacobian!#908

Closed
devmotion wants to merge 1 commit intomainfrom
devmotion-patch-1
Closed

Use FiniteDiff.finite_difference_jacobian!#908
devmotion wants to merge 1 commit intomainfrom
devmotion-patch-1

Conversation

@devmotion
Copy link
Copy Markdown
Member

Similar to the existing in-place methods that use finite_difference_gradient! and finite_difference_hessian!.

@devmotion devmotion requested a review from gdalle as a code owner November 5, 2025 10:26
@codecov
Copy link
Copy Markdown

codecov Bot commented Nov 5, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 43.62%. Comparing base (71bbd1c) to head (71eed5f).

Files with missing lines Patch % Lines
...xt/DifferentiationInterfaceFiniteDiffExt/onearg.jl 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (71bbd1c) and HEAD (71eed5f). Click for more details.

HEAD has 53 uploads less than BASE
Flag BASE (71bbd1c) HEAD (71eed5f)
DIT 12 6
DI 53 6
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #908       +/-   ##
===========================================
- Coverage   98.15%   43.62%   -54.54%     
===========================================
  Files         134      125        -9     
  Lines        8022     7847      -175     
===========================================
- Hits         7874     3423     -4451     
- Misses        148     4424     +4276     
Flag Coverage Δ
DI 30.64% <0.00%> (-68.21%) ⬇️
DIT 75.61% <ø> (-20.78%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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
Copy link
Copy Markdown
Member Author

Was looking at the wrong file.

@devmotion devmotion closed this Nov 5, 2025
@devmotion devmotion deleted the devmotion-patch-1 branch November 5, 2025 10:50
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.

1 participant