Skip to content
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

generators.py is required for the quickstart.py (from prophet.generators import YahooData) but is not included as part of the pip install. #22

Open
simongarisch opened this issue Sep 5, 2017 · 3 comments

Comments

@simongarisch
Copy link
Contributor

Hi. I've copied over the generators.py file for the quickstart to work, but figured it'd be easier if it was packaged as part of the default install.

@Emsu
Copy link
Owner

Emsu commented Sep 20, 2017

@simongarisch You mean the YahooData object? So you'd like it to be importable right?

@simongarisch
Copy link
Contributor Author

@Emsu Hi there. When running 'pip install prophet' from the command line prophet does not install the generators.py file which is required to import the YahooData class. The quickstart file, for example, requires that the YahooData class is available in order to run. It's just a minor thing, but it'd be good to update the files that available for install via pip.

@Emsu
Copy link
Owner

Emsu commented Oct 1, 2017

@simongarisch Would you mind adding that in a PR? Much appreciated!

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

No branches or pull requests

2 participants