-
Notifications
You must be signed in to change notification settings - Fork 849
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
joaozanlorensi/fixDocsTypo #836
joaozanlorensi/fixDocsTypo #836
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
Kindly check the flake8 output below:
|
Please advise me if any further tests or action on any of the other checklist items is needed. |
Thanks for the PR! Also, thanks for checking the flake8 output. In your case, since this was not due to new code you added, please don't worry about it. No further action required :) |
Description
This Pull Request aims to close issue #835. It consists of a small typo correction in the excellent documentation about the SequentialFeatureSelector.
Related issues or pull requests
Fixes #835.
Pull Request Checklist
./docs/sources/CHANGELOG.md
file (if applicable)./mlxtend/*/tests
directories (if applicable)mlxtend/docs/sources/
(if applicable)PYTHONPATH='.' pytest ./mlxtend -sv
and make sure that all unit tests pass (for small modifications, it might be sufficient to only run the specific test file, e.g.,PYTHONPATH='.' pytest ./mlxtend/classifier/tests/test_stacking_cv_classifier.py -sv
)flake8 ./mlxtend