-
-
Notifications
You must be signed in to change notification settings - Fork 55
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
simplified the steps in Readme file #208
Conversation
Signed-off-by: sumana sree <[email protected]>
Thank you for this. Would you be able to add a file back in to scripts/forecast_csv.py it would be copying this out to this serepate file |
Okay, I'll add the file. Thank you |
Signed-off-by: sumana sree <[email protected]>
Hi @sumana-2705. Sorry i might have confused you.
|
Signed-off-by: sumana sree <[email protected]>
I'm sorry @peterdudfield, I was confused a little. I hope this commit has done required changes |
and can you remove if name == "main": e.t.c from here |
Signed-off-by: sumana sree <[email protected]>
thanks @sumana-2705, @all-contributors please add @sumana-2705 for code |
I've put up a pull request to add @sumana-2705! 🎉 |
Description
Fixes #204
forecast_csv.py
script in the main README for better clarity.scripts
directory and then running the script to a single command.