Skip to content

Fix with_logabsdet_jacobian for mapped/broadcasted functions#40

Open
oschulz wants to merge 2 commits into
masterfrom
fix-mapped-noladj
Open

Fix with_logabsdet_jacobian for mapped/broadcasted functions#40
oschulz wants to merge 2 commits into
masterfrom
fix-mapped-noladj

Conversation

@oschulz

@oschulz oschulz commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Occurances of NoLogAbsDetJacobian in intermediate result were not handled properly.

@oschulz oschulz requested a review from devmotion July 8, 2026 20:12
@codecov

codecov Bot commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.74%. Comparing base (8bb9cfa) to head (393df8d).
⚠️ Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
src/with_ladj.jl 86.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   97.75%   95.74%   -2.01%     
==========================================
  Files           5        5              
  Lines          89       94       +5     
==========================================
+ Hits           87       90       +3     
- Misses          2        4       +2     

☔ View full report in Codecov by Harness.
📢 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.

@oschulz oschulz force-pushed the fix-mapped-noladj branch from f6b5d16 to 393df8d Compare July 8, 2026 20:32
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