-
Notifications
You must be signed in to change notification settings - Fork 60
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
Update 03_demand_forecasting.qmd with the base of a tutorial #75
Conversation
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.
It's a good start. I have a few updates to improve this.
Dataset: Please use walmart_sales dataset since its demand forecasting
Future Frame: Make use of the future frame function to create future dates by ID. This will allow us to show the future forecast.
Plotting: where possible use plot timeseries.
- Using wallmart dataset - Added usage of plot_timeseries
added the forecasting section, using future_frame
added doc'
I submitted a new version :) |
Ok thank you. Let me take a look. In the meantime I've approved checks to run. A few initial comments:
|
"dummies - I think this can be automated so dummies are created automatically in the augment function" do you mean I should add this change to the augment function itself and then make use of it ? "I want to see if anything else can be simplified to make it faster to go from data to results." I can try to see if I can simplify things a bit more. There are probably some cleaning possible. I wonder if the problem is not the Modelling section itself. I wanted to use the features from the dataset itself, but it does not really provide much improvement and complicates the code instead. Maybe I can just remove this part? |
This is a great start. I'm going to approve. I will have one of our core members take a look. Based on your workflow we will decide what additional functionality to invest time into and develop. Great work. |
Thank you very much, it was my pleasure :) |
Great work. 😀 |
A first solution for #48
closes #48