Skip to content

Accept hdi_prob in scCODA summaries with arviz 1.x - #1159

Merged
Zethson merged 1 commit into
mainfrom
fix/coda-arviz-ci-prob
Oct 2, 2026
Merged

Zethson merged 1 commit into
mainfrom
fix/coda-arviz-ci-prob

Conversation

@Zethson

@Zethson Zethson commented Oct 2, 2026

Copy link
Copy Markdown
Member

summary_prepare and summary accept hdi_prob again and forward it to arviz 1.x as ci_prob with ci_kind="hdi", and HDI columns are labelled HDI X% lower/upper like the ETI ones.
The documented sccoda.summary(mdata, hdi_prob=0.8) call no longer raises TypeError.

@codecov-commenter

codecov-commenter commented Oct 2, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.69%. Comparing base (72498c0) to head (8b34a20).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1159      +/-   ##
==========================================
+ Coverage   75.67%   75.69%   +0.02%     
==========================================
  Files          55       55              
  Lines        8431     8435       +4     
==========================================
+ Hits         6380     6385       +5     
+ Misses       2051     2050       -1     
Files with missing lines Coverage Δ
src/pertpy/tools/_coda/_base_coda.py 61.11% <100.00%> (+0.33%) ⬆️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Zethson
Zethson force-pushed the fix/coda-arviz-ci-prob branch from b50b967 to 4d30083 Compare October 2, 2026 11:02
arviz 1.x renamed the summary interval keyword from hdi_prob to ci_prob and names the columns hdi80_lb/hdi80_ub, so the documented hdi_prob= call raised a TypeError and HDI columns were never prettified.
summary_prepare now takes hdi_prob explicitly, forwards it as ci_prob with ci_kind="hdi", and the column renaming handles both HDI and ETI names.
Default output is unchanged and hdi_prob=0.8 matches ci_prob=0.8, ci_kind="hdi" exactly on haber_2017_regions.
@Zethson
Zethson force-pushed the fix/coda-arviz-ci-prob branch from 4d30083 to 8b34a20 Compare October 2, 2026 11:02
@Zethson
Zethson merged commit 0cd57fa into main Oct 2, 2026
22 checks passed
@Zethson
Zethson deleted the fix/coda-arviz-ci-prob branch October 2, 2026 11:21
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