Adds fit_params support for ExhaustiveFeatureSelector

[zdgriffith]

Description

Adds support for fit parameters to be passed to the fit method of ExhaustiveFeatureSelector.

Related issues or pull requests

Related to Pull Requests #255 , #350

Pull Request requirements

• Added appropriate unit test functions in the ./mlxtend/*/tests directories
• Ran nosetests ./mlxtend -sv and make sure that all unit tests pass
• Checked the test coverage by running nosetests ./mlxtend --with-coverage
• Checked for style issues by running flake8 ./mlxtend
• Added a note about the modification or contribution to the ./docs/sources/CHANGELOG.md file
• Modify documentation in the appropriate location under mlxtend/docs/sources/ (optional)
• Checked that the Travis-CI build passed at https://travis-ci.org/rasbt/mlxtend

[pep8speaks]

Hello @zdgriffith! Thanks for updating the PR.

• In the file mlxtend/feature_selection/sequential_feature_selector.py, following are the PEP8 issues :

Line 405:80: E501 line too long (84 > 79 characters)
Line 417:80: E501 line too long (81 > 79 characters)

Comment last updated on March 27, 2018 at 22:00 Hours UTC

[coveralls]

Coverage increased (+0.02%) to 91.746% when pulling d1ef89f on zdgriffith:efs-fit-params into c019c87 on rasbt:master.

[zdgriffith]

Sorry, I probably should have fit this into #350 .

[rasbt]

Thanks, Zach! And no worries, I think this way (having two PRs) makes it less confusing with regard to checking what's changed. The PR looks good to me, could you maybe also update the ExhaustiveFeatuerSelector.ipynb ? I.e., the last cell with the API documentation. You simply need to cd into mlxtend/docs, run python make_api.py and then run the last cell in the notebook

[zdgriffith]

Sorry about that, the notebook is updated now!

[rasbt]

Thanks a lot!