-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[WIP] Updated GP notebooks #4005
Conversation
Check out this pull request on Review Jupyter notebook visual diffs & provide feedback on notebooks. Powered by ReviewNB |
Awesome, thanks Chris! Going on holidays until July 17, so I'll be away from GH, but I'll be happy to review when I get back 😉 |
Codecov Report
@@ Coverage Diff @@
## master #4005 +/- ##
=======================================
Coverage 86.77% 86.77%
=======================================
Files 88 88
Lines 14137 14137
=======================================
Hits 12267 12267
Misses 1870 1870 |
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.
Thanks @fonnesbeck, almost all good -- just a nitpick: theano.tensor
is imported but unused so it can be dropped.
Also, I can't see the Bokeh plots in the updated version, but I suppose it's due to ReviewNB?
Ping @fonnesbeck . |
Wasn't quite ready to merge. Will open another PR, |
Sorry for jumping the gun.
…On Wed, Jul 29, 2020, 01:32 Chris Fonnesbeck ***@***.***> wrote:
Wasn't quite ready to merge. Will open another PR,
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#4005 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAFETGDJZ6PWJVY2SPE4KFTR55NYTANCNFSM4OTL32HQ>
.
|
This PR will fix the GP notebooks. Currently the first Mauna Loa notebook has been fixed; working on a couple of others.