Skip to content

Filter weighted samples before marginalization - #560

Merged
oschulz merged 2 commits into
mainfrom
fix-552-marginal-filtering
Sep 1, 2026
Merged

Filter weighted samples before marginalization#560
oschulz merged 2 commits into
mainfrom
fix-552-marginal-filtering

Conversation

@BJMCox

@BJMCox BJMCox commented Aug 16, 2026

Copy link
Copy Markdown
Member

Fixes #552.

Filter original weighted samples before projection so retained values and weights remain aligned. The 2D recipe avoids a second filter; zero-total weights retain prior behavior.

Main code: +5/-6.

@BJMCox BJMCox self-assigned this Aug 16, 2026
@codecov

codecov Bot commented Aug 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.45%. Comparing base (6e8795f) to head (04f51a8).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #560   +/-   ##
=======================================
  Coverage   65.45%   65.45%           
=======================================
  Files         121      121           
  Lines        7060     7060           
=======================================
  Hits         4621     4621           
  Misses       2439     2439           

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

@BJMCox
BJMCox force-pushed the fix-552-marginal-filtering branch 2 times, most recently from 1aacacf to 4228013 Compare September 1, 2026 12:06
@BJMCox
BJMCox force-pushed the fix-552-marginal-filtering branch from 4228013 to 04f51a8 Compare September 1, 2026 12:25
@BJMCox
BJMCox marked this pull request as ready for review September 1, 2026 12:34
@oschulz
oschulz merged commit b516b29 into main Sep 1, 2026
27 of 31 checks passed
@BJMCox
BJMCox deleted the fix-552-marginal-filtering branch September 4, 2026 07:31
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.

Apply low-weight filtering before marginal histograms

2 participants