Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Example Notebook for Users of 1.0 moving to 2.0 #299

Open
wants to merge 15 commits into
base: dev
Choose a base branch
from

Conversation

eodole
Copy link
Collaborator

@eodole eodole commented Feb 4, 2025

This is the first draft, I still need to make the validation section and have a few sections with ??? where there is a change that I feel should be explained but I'm not sure why that decision was made. I also have a section with additional features in which additonal features could be highlighted that are not included in this tutorial.
There are also several points in which is say "refer to the documentation" which I feel could be more specific but Im not sure if I should point to papers, or specific parts of the api or what exactly.

@eodole eodole requested a review from LarsKue February 4, 2025 15:31
Copy link
Contributor

@LarsKue LarsKue left a comment

Choose a reason for hiding this comment

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

The adapter changes made their way into this PR. @eodole can you make a new PR with just the notebook changes, please?

@eodole eodole force-pushed the example-update-notebook branch from bb0e0dc to 734e7c3 Compare February 13, 2025 19:33
@codecov-commenter
Copy link

codecov-commenter commented Feb 17, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines Coverage Δ
bayesflow/adapters/transforms/standardize.py 97.43% <ø> (+0.21%) ⬆️

... and 23 files with indirect coverage changes

…nter it made changes to the adapter formatting
@eodole eodole requested a review from stefanradev93 February 18, 2025 09:49
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.

3 participants