Skip to content

tutorial notebook for first users of soliket#221

Merged
ggalloni merged 14 commits intosimonsobs:masterfrom
MatForconi:tutorial_notebook
Feb 2, 2026
Merged

tutorial notebook for first users of soliket#221
ggalloni merged 14 commits intosimonsobs:masterfrom
MatForconi:tutorial_notebook

Conversation

@MatForconi
Copy link
Contributor

@MatForconi MatForconi commented Dec 18, 2025

Notebook for showing how to use soliket and mflike. Showing how to do runs with cobaya and what is inside the data.

  • Fix the path to the data file
  • Add comment for cell that install the likelihood
  • Update documentations

Copilot AI review requested due to automatic review settings December 18, 2025 10:32
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov-commenter
Copy link

codecov-commenter commented Dec 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.21%. Comparing base (61faa03) to head (5f77920).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #221      +/-   ##
==========================================
+ Coverage   83.79%   85.21%   +1.42%     
==========================================
  Files          35       33       -2     
  Lines        1641     1698      +57     
  Branches      146      164      +18     
==========================================
+ Hits         1375     1447      +72     
+ Misses        206      190      -16     
- Partials       60       61       +1     

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

@MatForconi
Copy link
Contributor Author

Ready to be merged @ggalloni

@ggalloni
Copy link
Collaborator

Hi @MatForconi , I've just merged the new rework of the GaussianLikelihood so I think that the notebook here needs to be updated to reflect the new way to handle data though sacc files. It should be as straight forward as going in the notebooks already present in the repo and copy-paste the relevant cells (for instance the ones where you read an hdu file).

Could you take care of this? Let me know 👍

@ggalloni ggalloni self-requested a review February 2, 2026 15:20
Copy link
Collaborator

@ggalloni ggalloni left a comment

Choose a reason for hiding this comment

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

@MatForconi, thx for taking care of this!

I am merging this as soon as the tests finish running.

@ggalloni ggalloni merged commit 5416da8 into simonsobs:master Feb 2, 2026
10 checks passed
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.

5 participants