Skip to content

Bayesian analysis#352

Open
rozyczko wants to merge 12 commits into
developfrom
bayesian
Open

Bayesian analysis#352
rozyczko wants to merge 12 commits into
developfrom
bayesian

Conversation

@rozyczko
Copy link
Copy Markdown
Member

@rozyczko rozyczko commented May 6, 2026

This pull request introduces an initial implementation steps for adding Bayesian MCMC (DREAM) sampling and analysis to EasyReflectometryLib, with a focus on a clean user-facing API and modular architecture.

The most important changes are:

Design and Implementation Plan

  • Added BAYESIAN_IN_ERL.md, a detailed implementation plan outlining the separation of classical optimization and Bayesian sampling APIs, the introduction of a sample() method for MCMC, new analysis modules, dependency requirements, and the testing strategy.

Analysis Module Creation

  • Created src/easyreflectometry/analysis/__init__.py to expose Bayesian analysis utilities, including PosteriorResults, plotting, summary, credible interval, and posterior predictive functions, making them available for import throughout the library.

Public API Enhancement

  • Updated src/easyreflectometry/__init__.py to export PosteriorResults in the library’s public API, ensuring that users can directly access Bayesian analysis results and utilities.

@rozyczko rozyczko added [scope] enhancement Adds/improves features (major.MINOR.patch) [priority] high Should be prioritized soon labels May 6, 2026
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pull request does not contain a valid label. Please add one of the following labels: ['chore', 'fix', 'bugfix', 'bug', 'enhancement', 'feature', 'dependencies', 'documentation']

@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2026

Codecov Report

❌ Patch coverage is 0% with 165 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (d773e7f) to head (d112d8d).

Files with missing lines Patch % Lines
src/easyreflectometry/analysis/bayesian.py 0.00% 136 Missing ⚠️
src/easyreflectometry/fitting.py 0.00% 20 Missing ⚠️
src/easyreflectometry/analysis/__init__.py 0.00% 8 Missing ⚠️
src/easyreflectometry/__init__.py 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #352    +/-   ##
========================================
  Coverage     0.00%   0.00%            
========================================
  Files           40      42     +2     
  Lines         2706    2871   +165     
========================================
- Misses        2706    2871   +165     
Flag Coverage Δ
integration 0.00% <0.00%> (ø)
unittests 0.00% <0.00%> (ø)

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

Files with missing lines Coverage Δ
src/easyreflectometry/__init__.py 0.00% <0.00%> (ø)
src/easyreflectometry/analysis/__init__.py 0.00% <0.00%> (ø)
src/easyreflectometry/fitting.py 0.00% <0.00%> (ø)
src/easyreflectometry/analysis/bayesian.py 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement [priority] high Should be prioritized soon [scope] enhancement Adds/improves features (major.MINOR.patch)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant