Skip to content

Add ForecastingTrack and change evalulate method#1764

Merged
kulbachcedric merged 9 commits intoonline-ml:mainfrom
kulbachcedric:1387-time-series-model-selection
Apr 1, 2026
Merged

Add ForecastingTrack and change evalulate method#1764
kulbachcedric merged 9 commits intoonline-ml:mainfrom
kulbachcedric:1387-time-series-model-selection

Conversation

@kulbachcedric
Copy link
Copy Markdown
Contributor

@kulbachcedric kulbachcedric commented Mar 25, 2026

This PR switches

from river.time_series.evaluate import _iter_with_horizon

to

from river.evaluate.forecasting import _iter_with_horizon

@kulbachcedric kulbachcedric linked an issue Mar 25, 2026 that may be closed by this pull request
# Conflicts:
#	docs/releases/unreleased.md
@kulbachcedric kulbachcedric self-assigned this Mar 25, 2026
@kulbachcedric kulbachcedric marked this pull request as ready for review March 25, 2026 10:03
@kulbachcedric kulbachcedric marked this pull request as draft March 25, 2026 10:04
@kulbachcedric kulbachcedric requested a review from e10e3 March 28, 2026 19:09
@kulbachcedric kulbachcedric marked this pull request as ready for review March 28, 2026 19:10
Copy link
Copy Markdown
Contributor

@e10e3 e10e3 left a comment

Choose a reason for hiding this comment

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

Good!
My comments are mostly nits.

I like how you handled the deprecation, with a test for the warning and all.

@kulbachcedric kulbachcedric merged commit 27b5029 into online-ml:main Apr 1, 2026
1 check 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.

2 participants