[ENH] Add general estimator test for common input dtypes #3137
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[ENH] Add general estimator test for common input dtypes
Reference Issues/PRs
dtypes#2803What does this implement/fix?
This PR introduces a new general estimator test to validate compatibility with common numpy input data types. The test converts the standard aeon testing data to the following dtypes and ensures the estimator continues to run as expected:
The new check executes
fitandpredict(where applicable) using the dtype-converted data and raises a clear failure if an estimator does not support any of these inputs. The check is integrated directly into the existing estimator testing flow.Does your contribution introduce a new dependency? If yes, which one?
No, this contribution does not introduce any new dependencies.
Any other comments?
PR checklist
For all contributions
For new estimators and functions
For developers with write access