-
-
Notifications
You must be signed in to change notification settings - Fork 25k
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
[MRG] Add explanation of why iterative imputer is experimental #17115
Conversation
Reference current issues needed to be resolved before class can potentially be stabilized.
doc/modules/impute.rst
Outdated
you need to explicitly import ``enable_iterative_imputer``. | ||
This estimator is still **experimental** for now: default parameters or | ||
details of behaviour might change without any deprecation cycle, | ||
specifically convergence criteria (`issue 14338 <https://github.com/scikit-learn/scikit-learn/issues/14338>`_), |
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.
specifically convergence criteria (`issue 14338 <https://github.com/scikit-learn/scikit-learn/issues/14338>`_), | |
specifically convergence criteria (:issue:`14338`), |
You can use these shortcuts from the Sphinx-issues extension
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.
TIL. Done.
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.
When the issues gets resolved, we would need to remember to remove them from the paragraph.
doc/modules/impute.rst
Outdated
This estimator is still **experimental** for now: default parameters or | ||
details of behaviour might change without any deprecation cycle, | ||
specifically convergence criteria (:issue:`14338`), | ||
default estimators (:issue:`13286`), | ||
and use of random state (:issue:`15611`). | ||
To use it, you need to explicitly import ``enable_iterative_imputer``. |
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.
This can be broken up into two sentences:
This estimator is still experimental for now: default parameters or details of behavior might change without any deprecation cycle. Resolving the following issues would help stabilize :class:
IterativeImputer
: ...
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.
Updated.
Thanks @skeller88 |
Fixes #16638 by referencing current issues needed to be resolved before class can potentially be stabilized. That makes it clearer to potential users of this class what the current unknowns are, and makes it easier for potential contributors to help with these issues.