Skip to content
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

Minor rephrasing in the combine doc #330

Merged
merged 2 commits into from
Aug 25, 2017

Conversation

rasbt
Copy link
Contributor

@rasbt rasbt commented Aug 25, 2017

Reference Issue

What does this implement/fix? Explain your changes.

I just noticed the new release and was reading through the new documentation, which is really awesome! While reading, I found the description on combining under- and over-sampling methods a bit hard to read due to some minor typos and though rephrasing it a bit would help.

Any other comments?

@codecov
Copy link

codecov bot commented Aug 25, 2017

Codecov Report

Merging #330 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #330   +/-   ##
=======================================
  Coverage   98.25%   98.25%           
=======================================
  Files          68       68           
  Lines        4299     4299           
=======================================
  Hits         4224     4224           
  Misses         75       75

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 96b4c31...29f0b5c. Read the comment docs.

@glemaitre
Copy link
Member

Thanks a lot!!!

@glemaitre glemaitre merged commit 3e0f3cc into scikit-learn-contrib:master Aug 25, 2017
@massich
Copy link
Contributor

massich commented Aug 28, 2017

@rasbt Big kudos !!!! thanks.

glemaitre pushed a commit to glemaitre/imbalanced-learn that referenced this pull request Oct 9, 2017
* minor rephrasing in the combine doc

* Update combine.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants