Skip to content

Improved HDI computation#127

Open
arossia94 wants to merge 15 commits into
mainfrom
Improved_HDIs
Open

Improved HDI computation#127
arossia94 wants to merge 15 commits into
mainfrom
Improved_HDIs

Conversation

@arossia94
Copy link
Copy Markdown
Collaborator

@arossia94 arossia94 commented Mar 26, 2025

Improved computation of HDIs with variation of confidence level and reporting several disjoint intervals and the peak of the distribution inside each interval.

Changes and improvements:

  • HDIs computed with different confidence levels.
  • Better reporting of disjoint HDIs.
  • Computation and reporting of the 1st mode inside each disjoint interval.
  • Computation of monomodal HDIs.
  • Option in the report runcard to use multimodal or unimodal HDIs.
  • Fix the spider plot
  • Add instructions and examples in the docs.

Improved computation of HDIs with variation of confidence level and reporting several disjoint intervals and the peak of the distribution inside each interval.
@LucaMantani
Copy link
Copy Markdown
Collaborator

Could you add an example on how to obtain these bounds in the report, if any new option is available in the card, etc

@arossia94
Copy link
Copy Markdown
Collaborator Author

@jacoterh , when you have time, could you take a look at the spider plot function here? Somehow it broke down but I have no idea why.

@arossia94
Copy link
Copy Markdown
Collaborator Author

The spider plot is working now, but requires more thorough testing.

@arossia94 arossia94 requested a review from jacoterh September 25, 2025 13:20
@arossia94
Copy link
Copy Markdown
Collaborator Author

The module coefficients_utils.py needs a deep restructuring to pass the CodeFactor check.
In particular, we could move the internal part of write_cl_table and some of the functions inside the class CoefficientsPlotter to a separate module (even perhaps that whole class...).

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