[MRG+1] Parallelisation for SMOTEENN and SMOTETomek

[Microsheep]

Fixes #541

This Merge Request adds n_jobs parameter for SMOTEENN and SMOTETomek

[pep8speaks]

Hello @Microsheep! Thanks for updating the PR.

Cheers ! There are no PEP8 issues in this Pull Request. 🍻

Comment last updated on February 19, 2019 at 13:10 Hours UTC

[codecov]

Codecov Report

Merging #547 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #547      +/-   ##
==========================================
+ Coverage   98.86%   98.87%   +<.01%     
==========================================
  Files          82       82              
  Lines        5025     5049      +24     
==========================================
+ Hits         4968     4992      +24     
  Misses         57       57

Impacted Files	Coverage Δ
imblearn/combine/_smote_tomek.py	100% <100%> (ø)	⬆️
imblearn/combine/_smote_enn.py	100% <100%> (ø)	⬆️
imblearn/combine/tests/test_smote_tomek.py	100% <100%> (ø)	⬆️
imblearn/combine/tests/test_smote_enn.py	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4559071...aba9470. Read the comment docs.

[glemaitre]

Thanks can you add an entry in whats new and a test.

For the test, you can just set n_jobs and check that the underlying enn or smote we have the expected number of jobs.

[Microsheep]

@glemaitre Thanks for the comment, I had finish adding tests and added an entry in whats new.

[glemaitre]

LGTM @chkoar could you have a second look at it

[chkoar]

@Microsheep in the issue number put the number of your PR. Apart from that it's ok.

[Microsheep]

@chkoar Got it! I had change to use the PR number.

[glemaitre]

@Microsheep Thanks for the PR!!!