[MatthewMiddlehurst]

Reference Issues/PRs

Follows a review of the estimator getters. The clusterer and forecaster getters are not changed here.

What does this implement/fix? Explain your changes

Fixes names in get_classifier_by_name and get_regressor_by_name which ignore an argument or build something other than the name says.

Getter	Names	Problem	Fix
regressor	1nn-ed, 5nn-ed	Built with the KNeighborsTimeSeriesRegressor default distance, which is dtw, so both ran full-window DTW	Set distance="euclidean"
regressor	1nn-msm, 5nn-msm	kwargs were not passed on. Valid ones were ignored and invalid ones accepted	Pass kwargs
regressor	all six kNN names	n_jobs was not passed on	Pass n_jobs
classifier	rdst, 1nn-dtw-cv	n_jobs was not passed on	Pass n_jobs (to GridSearchCV for 1nn-dtw-cv, as grid-svr does)
classifier	logistic	n_jobs was passed to LogisticRegression, where it has no effect since scikit-learn 1.8 and warns in fit until its removal in 1.10	Stop passing n_jobs
both	tsf, rise, cif and their -500 versions, plus classifier stsf, stsf-500	fit_contract was not passed on, so a contract was ignored. drcif already passes it	Pass time_limit_in_minutes=fit_contract

Needs careful review:

• 1nn-ed and 5nn-ed now give different results. Earlier results from get_regressor_by_name under these names are DTW. The TSER archive publication has its own set_tser_exp_regressor.py, which already sets distance="euclidean", so its results are not affected.
• A contract now applies to the interval forests. With fit_contract above 0 aeon ignores n_estimators and builds until the time limit or contract_max_n_estimators (500), the same as drcif. The default of 0 is unchanged: no contract and n_estimators is used.
• logistic no longer sets n_jobs on any scikit-learn version. Before 1.8 it only applied to one-vs-rest fits.

Any other comments?

No tests are added. Run locally with Python 3.13.9, aeon 1.6.0 and scikit-learn 1.9.1 (all_extras installed):

• python -m pytest -n logical tsml_eval/experiments: 67 passed
• pre-commit run --all-files: passed
• mypy tsml_eval/: no issues

Contract-capable names left as they are: stc-2hour, where the name sets a transform limit, and regressor tsf-i, where the forest is built per channel inside an ensemble.

[tsml-actions-bot]

Thank you for contributing to tsml-eval

I have added the following labels to this PR based on the title: .
I have added the following labels to this PR based on the changes made: . Feel free to change these if they do not properly represent the PR.

The Checks tab will show the status of our automated tests. You can click on individual test runs in the tab or "Details" in the panel below to see more information if there is a failure.

If our pre-commit code quality check fails, please run pre-commit locally and push the fixes to your PR branch.

PR CI actions

These checkboxes will add labels to enable or disable CI functionality for this PR. This may not take effect immediately, and a new commit may be required to run the new configuration.

• Run pre-commit checks for all files
• Run all pytest tests and configurations
• Run all notebook example tests
• Push an empty commit to re-run CI checks