Skip to content

Predict Next Purchase: Download Data#76

Merged
jeff-hernandez merged 11 commits into
masterfrom
predict-next-purchase
Apr 14, 2020
Merged

Predict Next Purchase: Download Data#76
jeff-hernandez merged 11 commits into
masterfrom
predict-next-purchase

Conversation

@jeff-hernandez

@jeff-hernandez jeff-hernandez commented Sep 19, 2019

Copy link
Copy Markdown
Contributor

This will make the doc example for Predict Next Purchase more user friendly by automatically downloading the data if it does not already exist. The notebook is hosted here. Closes #59.

@codecov

codecov Bot commented Sep 19, 2019

Copy link
Copy Markdown

Codecov Report

Merging #76 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #76   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           20        20           
  Lines          820       820           
=========================================
  Hits           820       820           

@CLAassistant

CLAassistant commented Sep 27, 2019

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@jeff-hernandez jeff-hernandez requested a review from rwedge November 1, 2019 15:56
@kmax12

kmax12 commented Apr 14, 2020

Copy link
Copy Markdown
Contributor

@jeff-hernandez what's the status of this?

@jeff-hernandez

Copy link
Copy Markdown
Contributor Author

@kmax12 this should be ready for review. Let me update with the base branch for confirmation.

@jeff-hernandez

Copy link
Copy Markdown
Contributor Author

Yes, this is ready for review.

@kmax12 kmax12 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@jeff-hernandez jeff-hernandez merged commit 6814e26 into master Apr 14, 2020
@jeff-hernandez jeff-hernandez deleted the predict-next-purchase branch April 14, 2020 21:18
@jeff-hernandez jeff-hernandez mentioned this pull request Apr 23, 2020
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.

Update Predict Next Purchase

3 participants