* Added value checks and random state parameter to method
* Changed default parameter to None instead of 0
* Added numpy RandomState to the check
* Replaced inline validation with check_random_state from utils and pointed at glossery
* Needed a different default parameter to pass the default way this has been working in the past
* Updated to conform with flake8 stds
* Add random_state to AffinityPropagation class.
* Add test.
* Add what's new entry and versionadded directive.
* Add PR number.
* Fix lint error due to this PR.
* Use np.array_equal in test.
* Update sklearn/cluster/_affinity_propagation.py
Co-Authored-By: Adrin Jalali <adrin.jalali@gmail.com>
* Homogenize parametre descriptions, default random_state to None.
* Update sklearn/cluster/_affinity_propagation.py
Co-Authored-By: Nicolas Hug <contact@nicolas-hug.com>
* Update sklearn/cluster/_affinity_propagation.py
Co-Authored-By: Nicolas Hug <contact@nicolas-hug.com>
* Update sklearn/cluster/_affinity_propagation.py
Co-Authored-By: Nicolas Hug <contact@nicolas-hug.com>
* Update doc/whats_new/v0.23.rst
Co-Authored-By: Nicolas Hug <contact@nicolas-hug.com>
* Change test name.
* Modify check in test.
* Fix lint errors.
* Address comment.
* Address comment.
* Add 'deprecation' and its correspondent test.
* Fix lint errors.
* Add random_state parameter to tests, to avoid FutureWarnings.
* Move warning in fit. Modify tests.
* Modify example.
* Tentative fix for failures.
* Document default value to 0. Revert docstring.
* Explicit link to Glossary.
* Fix default value.
* Remove some warnings from tests.
* Validate and test docstring.
* Tentative fix.
* Tentative fix.
* Ignore FutureWarning in fit attribute test.
* Set random_state to avoid FutureWarning in test_fit_docstring_attributes.
* [doc build] Force documentation build.
* Clarify warning message.
Co-authored-by: rwoolston.admin <rwoolston.admin@LXO-DS-DEV.afcucorp.local>
Co-authored-by: Adrin Jalali <adrin.jalali@gmail.com>
Co-authored-by: Nicolas Hug <contact@nicolas-hug.com>
Previously error messages for mismatches parameters on function/method
signatures and docstrings were not explicit or user-friendly. This
changes provides a more explicit error message and in addition notifies
the relation of missing/mismatching parameters from signature to
docstring and from docstring to signature.
* add docstring tests to a travis entry that actually runs tests
* show skipped tests
* better test skipping messages
* use path in walk_packages so we can run the tests from anywhere.
Also try to do better tests for private packages.
* Ensure all submodule classes and functions are tested
* Reverse the for loop nesting to avoid copying
* skip abstract methods, skip setup.configure, skip a lot more that I don't want to fix.
* unused import
* move neighbors up from deprecated to just not covered.
Jie Zheng
Hanmin Qin