search

tensorflow / skflow

Simplified interface for TensorFlow (mimicking Scikit Learn) for Deep Learning
Apache License 2.0
3.18k stars 439 forks source link

DaskDataFeeder (fixes issue #53) #59

Closed terrytangyuan closed 8 years ago

terrytangyuan commented 8 years ago

Fixes #53

Will add test for this in a separate PR. Some refactoring is needed later too.

This PR is for reviewing of possible grammatical optimization/mistakes and architecture.

codecov-io commented 8 years ago

Current coverage is 92.37%

Merging #59 into master will decrease coverage by -2.61% as of e05b4a0

@@            master     #59   diff @@
======================================
  Files           31      32     +1
  Stmts          957    1010    +53
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit            909     933    +24
  Partial          0       0       
- Missed          48      77    +29

Review entire Coverage Diff as of e05b4a0

Powered by Codecov. Updated on successful CI builds.

terrytangyuan commented 8 years ago

@ilblackdragon Thanks! I've pushed the changes according to your comments.