Skip to content

DOC effect and comparison to deal with imbalanced classification #648

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

Merged
merged 12 commits into from
Nov 19, 2019

Conversation

glemaitre
Copy link
Member

This will show the effects of dealing with imbalanced classes.
In addition, we will show the different strategy which comes out-of-the-box in scikit-learn and the couple of methods which could be used successively from imbalanced-learn

@pep8speaks
Copy link

pep8speaks commented Nov 17, 2019

Hello @glemaitre! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 22:1: E402 module level import not at top of file
Line 42:1: E402 module level import not at top of file
Line 57:1: E402 module level import not at top of file
Line 67:1: E402 module level import not at top of file
Line 77:1: E402 module level import not at top of file
Line 92:1: E402 module level import not at top of file
Line 137:1: E402 module level import not at top of file
Line 138:1: E402 module level import not at top of file
Line 139:1: E402 module level import not at top of file
Line 140:1: E402 module level import not at top of file
Line 155:1: E402 module level import not at top of file
Line 156:1: E402 module level import not at top of file
Line 157:1: E402 module level import not at top of file
Line 169:1: E402 module level import not at top of file
Line 186:1: E402 module level import not at top of file
Line 187:1: E402 module level import not at top of file
Line 252:1: E402 module level import not at top of file
Line 253:1: E402 module level import not at top of file
Line 300:1: E402 module level import not at top of file
Line 315:1: E402 module level import not at top of file
Line 316:1: E402 module level import not at top of file
Line 317:1: E402 module level import not at top of file

Comment last updated at 2019-11-19 22:36:37 UTC

@lgtm-com
Copy link

lgtm-com bot commented Nov 17, 2019

This pull request introduces 3 alerts when merging 96ef7d4 into 158258e - view on LGTM.com

new alerts:

  • 1 for Wrong name for an argument in a call
  • 1 for Mismatch in multiple assignment
  • 1 for Unused import

@codecov
Copy link

codecov bot commented Nov 17, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #648   +/-   ##
=======================================
  Coverage   98.46%   98.46%           
=======================================
  Files          82       82           
  Lines        4873     4873           
=======================================
  Hits         4798     4798           
  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 f356284...8b11705. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 3 alerts when merging 2940dbb into f356284 - view on LGTM.com

new alerts:

  • 2 for Unused import
  • 1 for Wrong name for an argument in a call

Co-Authored-By: Christos Aridas <chkoar@users.noreply.github.com>
@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 2 alerts and fixes 3 when merging 4d0a629 into f356284 - view on LGTM.com

new alerts:

  • 2 for Unused import

fixed alerts:

  • 2 for Mismatch in multiple assignment
  • 1 for Wrong name for an argument in a call

@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 2 alerts when merging db914bf into f356284 - view on LGTM.com

new alerts:

  • 1 for Wrong name for an argument in a call
  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 2 alerts when merging 802cc44 into f356284 - view on LGTM.com

new alerts:

  • 1 for Wrong name for an argument in a call
  • 1 for Unused import

@glemaitre glemaitre merged commit bbbf0e2 into scikit-learn-contrib:master Nov 19, 2019
@lgtm-com
Copy link

lgtm-com bot commented Nov 19, 2019

This pull request introduces 2 alerts when merging 8b11705 into f356284 - view on LGTM.com

new alerts:

  • 1 for Wrong name for an argument in a call
  • 1 for Unused import

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.

3 participants