-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
DOC Ensures plot_precision_recall_curve passes the numpydoc validation #24403
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
Conversation
036c4db
to
945d666
Compare
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.
We have the same issue as for other plotting functions that are deprecated. We need to keep the function in the white list.
@@ -18,7 +18,6 @@ | |||
# sklearn.deprecation._update_doc is updating the doc against the numpydoc. | |||
# This will be fixed with issue: #24328 | |||
"sklearn.metrics._plot.det_curve.plot_det_curve", |
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.
"sklearn.metrics._plot.det_curve.plot_det_curve", | |
"sklearn.metrics._plot.det_curve.plot_det_curve", | |
# sklearn.deprecation._update_doc is updating the doc against the numpydoc. | |
# See gh-24328. | |
"sklearn.metrics._plot.precision_recall_curve.plot_precision_recall_curve", |
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.
added the comment
@@ -417,12 +417,6 @@ def plot_precision_recall_curve( | |||
|
|||
Read more in the :ref:`User Guide <precision_recall_f_measure_metrics>`. | |||
|
|||
.. deprecated:: 1.0 |
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.
Can you revert this change
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.
done
90f7773
to
013486c
Compare
013486c
to
a1fbe77
Compare
Thanks. I will merge once the CIs are green. |
Addresses #21350
This PR ensures plot_precision_recall_curve is compatible with numpydoc:
What does this implement/fix? Explain your changes.
In #21547, and #22247 it was discussed about removing the depreciated docstrings to avoid the
GL09
warning