search

cesium-ml / cesium

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

Do not pass all_time_series to assemble_featureset in featurize.py #251

Closed acrellin closed 7 years ago

pep8speaks commented 7 years ago

Hello @acrellin! Thanks for updating the PR.

Line 295:81: E501 line too long (83 > 80 characters)

Comment last updated on June 16, 2017 at 21:57 Hours UTC
codecov-io commented 7 years ago

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #251   +/-   ##
=========================================
  Coverage          ?   92.62%           
=========================================
  Files             ?       35           
  Lines             ?     1858           
  Branches          ?      237           
=========================================
  Hits              ?     1721           
  Misses            ?      100           
  Partials          ?       37
Impacted Files Coverage Δ
cesium/featurize.py 97.36% <100%> (ø)

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 0300c8f...c78f328. Read the comment docs.