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

Fixed minor issues with Google stock time series forecasting notebook #427

Merged
merged 1 commit into from Sep 2, 2018

Conversation

Projects
None yet
3 participants
@WillKoehrsen
Contributor

WillKoehrsen commented Aug 31, 2018

Addresses #426 by fixing a few minor issues that prevented the timeseries_forecasting_google_stock Jupyter Notebook from successfully running. No major changes to the content were made, only those required for the notebook to run.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 31, 2018

Coverage Status

Coverage remained the same at 97.444% when pulling 6bb4335 on WillKoehrsen:google-stock-notebook-fixes into 8a790ff on blue-yonder:master.

coveralls commented Aug 31, 2018

Coverage Status

Coverage remained the same at 97.444% when pulling 6bb4335 on WillKoehrsen:google-stock-notebook-fixes into 8a790ff on blue-yonder:master.

@MaxBenChrist

This comment has been minimized.

Show comment
Hide comment
@MaxBenChrist

MaxBenChrist Sep 2, 2018

Collaborator

Thanks for the pr @WillKoehrsen!

Collaborator

MaxBenChrist commented Sep 2, 2018

Thanks for the pr @WillKoehrsen!

@MaxBenChrist MaxBenChrist merged commit 7933c4b into blue-yonder:master Sep 2, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 97.444%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment