-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add Forest dashboard #16
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
We need to update the notebook to save the log likelihood in the inference data object. We should also pin panel in the requirements file to be less than 1 so we get the right version of bokeh installed. @yilinxia I think we can merge after those changes. |
@ndmlny-qs Hi Andy, I have addressed the issues you pointed out. Meanwhile, I am also thinking about maybe we should use the same standard to “beautify” the code I wrote before (in another branch?) |
I agree, we need to have steps that incorporate |
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
@aloctavodia Hi Osvaldo, do you have any other comments? otherwise, I think we can merge this branch |
add forest_dashboard including