-
Notifications
You must be signed in to change notification settings - Fork 12
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
Improve getting started tutorial #634
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
leaving a small comment.
If this is the intended scope of the PR, then otherwise good to merge.
"\n", | ||
"If `model` provided is not one of the above, then HSSM considers that you are using a custom model. We will cover custom models in more detail in a separate tutorial.\n", | ||
"For custom type modeling, please use the `model` parameter of the HSSM class constructor (details in the [creating custom models](/docs/getting_started/hierarchical_modeling.ipynb) tutorial).\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this the right/intended tutorial that is linked here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think so (?). The original text says "We will cover custom models in more detail in a separate tutorial." I assumed that separate tutorial is the one in the /docs/getting_started/hierarchical_modeling.ipynb
notebook. Is that correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this https://lnccbrown.github.io/HSSM/tutorials/jax_callable_contribution_onnx_example/ and this https://lnccbrown.github.io/HSSM/tutorials/blackbox_contribution_onnx_example/ might work a bit better
Codecov ReportAll modified and coverable lines are covered by tests ✅ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Improve clarity; link related tutorials.