search

cesium-ml / cesium

Machine Learning Time-Series Platform
Other
670 stars 101 forks source link

`Featureset` API tweaks #236

Closed bnaul closed 7 years ago

bnaul commented 7 years ago
codecov-io commented 7 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@21e5543).

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 21e5543...bc0b141. Read the comment docs.

codecov-io commented 7 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9976707). Click here to learn what that means. The diff coverage is 93.33%.

@@            Coverage Diff            @@
##             master     #236   +/-   ##
=========================================
  Coverage          ?   94.12%           
=========================================
  Files             ?       43           
  Lines             ?     2196           
  Branches          ?      272           
=========================================
  Hits              ?     2067           
  Misses            ?       73           
  Partials          ?       56
Impacted Files Coverage Δ
cesium/time_series.py 83.33% <ø> (ø)
cesium/featureset.py 92.68% <100%> (ø)
cesium/tests/test_featureset.py 100% <100%> (ø)
cesium/tests/fixtures.py 97.77% <100%> (ø)
cesium/featurize.py 94.89% <83.33%> (ø)

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 9976707...c84b6b7. Read the comment docs.

bnaul commented 7 years ago

@stefanv Need to do a release, can we go ahead and merge this? Not sure what Codecov is going on about, I fixed exactly 1 thing and added a test for it..

stefanv commented 7 years ago

Looks good.