Skip to content

gh-142: Update BAO likelihood notebook to use new BAO formats#143

Open
chiaramoretti wants to merge 2 commits into
mainfrom
142-update-bao-likelihood
Open

gh-142: Update BAO likelihood notebook to use new BAO formats#143
chiaramoretti wants to merge 2 commits into
mainfrom
142-update-bao-likelihood

Conversation

@chiaramoretti
Copy link
Copy Markdown
Member

@chiaramoretti chiaramoretti commented Apr 24, 2026

📌 Related Issue

Closes #142

✨ What’s been added or changed?

BAO likelihood tutorial updated with new formats of the datamodel

🧪 How was it tested?

Tested locally, both in this version and with a simple 2 parameters sampling

🔍 Checklist

You can review this PR in ReviewNB, don't forget to log in with your GitHub username:
👉 https://app.reviewnb.com/cloe-org/playground/pulls/

  • I've named the title of this pull request starting by 'gh-#: TITLE'
  • I’ve linked the related issue
  • I’ve tested the code or notebooks manually
  • I’ve added comments/docstrings where needed (i.e: README)
  • I’ve updated relevant docs if necessary

@chiaramoretti chiaramoretti self-assigned this Apr 24, 2026
@chiaramoretti chiaramoretti added the bug Something isn't working label Apr 24, 2026
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@gcanasherrera gcanasherrera self-requested a review April 24, 2026 13:33
@gcanasherrera
Copy link
Copy Markdown
Member

gcanasherrera commented Apr 24, 2026

@chiaramoretti conflicts must be fixed.

@chiaramoretti
Copy link
Copy Markdown
Member Author

@gcanasherrera should be fixed, it was mainly metadata or cells outputs

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

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update BAO likelihood notebook

2 participants