Skip to content

docs: fix undefined variable and comment mismatch in time series forecast example - #1580

Merged
Li Jiang (thinkall) merged 4 commits into
microsoft:mainfrom
MeiSiristhebest:fix/ts-forecast-doc-bugs
Aug 14, 2026
Merged

docs: fix undefined variable and comment mismatch in time series forecast example#1580
Li Jiang (thinkall) merged 4 commits into
microsoft:mainfrom
MeiSiristhebest:fix/ts-forecast-doc-bugs

Conversation

@MeiSiristhebest

Copy link
Copy Markdown
Contributor

Fixes #1173 and #1174 (part of #1570).

In website/docs/Examples/AutoML-Time series forecast.md:

  • Fix NameError in multivariate forecasting example: update dataframe=df to dataframe=multi_train_df (the DataFrame created in preceding steps).
  • Fix comment mismatch: update # remove temperature column to reduce redundancy to # remove month column to reduce redundancy.

Copilot AI 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.

Pull request overview

Fixes two errors in the multivariate forecasting documentation example.

Changes:

  • Corrects the deleted-column comment.
  • Uses the defined multivariate training DataFrame.
Suppressed comments (1)

website/docs/Examples/AutoML-Time series forecast.md:534

  • The multivariate split reserves 180 rows via multi_time_horizon, but this still passes the earlier univariate example's 12-row time_horizon. This tunes/validates for the wrong forecast length and differs from the matching notebook example; pass multi_time_horizon here.
automl.fit(dataframe=multi_train_df, **settings, period=time_horizon)

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread website/docs/Examples/AutoML-Time series forecast.md
@MeiSiristhebest

Copy link
Copy Markdown
Contributor Author

Updated notebook/automl_time_series_forecast.ipynb comment to match the deleted month column and fixed period=multi_time_horizon in website/docs/Examples/AutoML-Time series forecast.md to align with the notebook example. Ready for review!

Copilot AI 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.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@thinkall
Li Jiang (thinkall) merged commit 53e3461 into microsoft:main Aug 14, 2026
13 checks passed
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.

error in example for multivariate-time-series-forecasting-with-exogenous-variables

3 participants