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

EHN add support for some Keras utilities #409

Merged
merged 55 commits into from Aug 22, 2018

Conversation

Projects
None yet
4 participants
@glemaitre
Member

glemaitre commented Mar 1, 2018

This PR attend to provide some utilities for keras:

  • support for one-vs-all encoded targets (#410)
  • balanced batch generator

TODO:

  • Add common test to check multiclass == multilabel-indicator (#410)
  • Manage the specificity of the EasyEnsemble and BalanceCascade (overwrite sample)
  • Add user guide documentation
  • Add an example for simple use
  • Add an example for deep training
  • Add substitution
  • What's new
  • Optional depencies

glemaitre added some commits Mar 1, 2018

@codecov

This comment has been minimized.

codecov bot commented Mar 1, 2018

Codecov Report

Merging #409 into master will increase coverage by 0.01%.
The diff coverage is 99.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #409      +/-   ##
==========================================
+ Coverage   98.75%   98.77%   +0.01%     
==========================================
  Files          70       75       +5     
  Lines        4268     4410     +142     
==========================================
+ Hits         4215     4356     +141     
- Misses         53       54       +1
Impacted Files Coverage Δ
imblearn/utils/_docstring.py 100% <ø> (ø) ⬆️
imblearn/keras/__init__.py 100% <100%> (ø)
imblearn/tensorflow/__init__.py 100% <100%> (ø)
imblearn/tensorflow/_generator.py 100% <100%> (ø)
imblearn/keras/tests/test_generator.py 100% <100%> (ø)
imblearn/keras/_generator.py 98.18% <98.18%> (ø)
... and 1 more

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 eafae67...abeb011. Read the comment docs.

@pep8speaks

This comment has been minimized.

pep8speaks commented Mar 1, 2018

Hello @glemaitre! Thanks for updating the PR.

Line 27:1: E402 module level import not at top of file
Line 28:1: E402 module level import not at top of file
Line 29:1: E402 module level import not at top of file
Line 51:1: E402 module level import not at top of file
Line 52:1: E402 module level import not at top of file
Line 53:1: E402 module level import not at top of file
Line 54:1: E402 module level import not at top of file
Line 55:1: E402 module level import not at top of file
Line 56:1: E402 module level import not at top of file
Line 86:1: E402 module level import not at top of file
Line 93:1: E402 module level import not at top of file
Line 94:1: E402 module level import not at top of file
Line 128:1: E402 module level import not at top of file
Line 129:1: E402 module level import not at top of file
Line 148:1: E402 module level import not at top of file
Line 163:1: E402 module level import not at top of file
Line 185:1: E402 module level import not at top of file
Line 220:1: E402 module level import not at top of file
Line 221:1: E402 module level import not at top of file

Line 9:1: E402 module level import not at top of file
Line 10:1: E402 module level import not at top of file
Line 11:1: E402 module level import not at top of file
Line 13:1: E402 module level import not at top of file
Line 14:1: E402 module level import not at top of file
Line 15:1: E402 module level import not at top of file
Line 17:1: E402 module level import not at top of file
Line 18:1: E402 module level import not at top of file

Comment last updated on August 22, 2018 at 09:46 Hours UTC

glemaitre added some commits Apr 4, 2018

@glemaitre

This comment has been minimized.

Member

glemaitre commented Apr 6, 2018

@jorisvandenbossche I think that it could be of interest. Let me know if the API make sense to you.

glemaitre added some commits Apr 6, 2018

except ImportError:
# Skip the tests for the examples
import pytest
keras = pytest.importorskip('keras')

This comment has been minimized.

@chkoar

chkoar Apr 7, 2018

Member

This line in my box generates the following error:

Skipped: could not import 'keras'

This comment has been minimized.

@glemaitre

glemaitre Apr 7, 2018

Member

I wanted to use it to skip the test of the docstring but probably there is something better to do then.

@glemaitre

This comment has been minimized.

Member

glemaitre commented Apr 7, 2018

Otherwise does it look ok

glemaitre added some commits May 14, 2018

@glemaitre glemaitre force-pushed the scikit-learn-contrib:master branch from 155fe0f to f4271d9 May 14, 2018

@glemaitre glemaitre force-pushed the glemaitre:keras_batch_generator branch from 8c4c344 to c056567 Aug 21, 2018

glemaitre added some commits Aug 21, 2018

@glemaitre glemaitre force-pushed the glemaitre:keras_batch_generator branch from 765dc8a to 324b90c Aug 22, 2018

glemaitre added some commits Aug 22, 2018

@glemaitre glemaitre merged commit 7c48491 into scikit-learn-contrib:master Aug 22, 2018

6 checks passed

LGTM analysis: Python 2 new alerts
Details
ci/circleci: python3 Your tests passed on CircleCI!
Details
codecov/patch 99.29% of diff hit (target 98.75%)
Details
codecov/project 98.77% (+0.01%) compared to eafae67
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment