Closed pbloem closed 2 years ago
Hello @pbloem! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Great, thanks! I don't think we'll need a doc example for that, so it should be good to merge.
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
Code of Conduct
Check.
Description
I added a kwargs keyword to control how the lines between decision regions look. I needed this for a thing, so I thought I'd make it a pull request. I think I followed the logic of the other two kwargs arguments correctly.
Related issues or pull requests
None
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