search

cesium-ml / cesium

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

added bookkeeping to the results dictionary #268

Closed profjsb closed 1 year ago

profjsb commented 6 years ago

added two new key/values based on the original L-S refine script.

pep8speaks commented 6 years ago

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

Line 6:81: E501 line too long (93 > 80 characters) Line 35:1: W293 blank line contains whitespace Line 291:47: E226 missing whitespace around arithmetic operator Line 291:51: W504 line break after binary operator Line 293:45: E128 continuation line under-indented for visual indent Line 293:69: E226 missing whitespace around arithmetic operator

Comment last updated at 2020-11-14 03:36:17 UTC
codecov-io commented 6 years ago

Codecov Report

Merging #268 into master will increase coverage by <.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
+ Coverage   93.11%   93.12%   +<.01%     
==========================================
  Files          34       34              
  Lines        1903     1905       +2     
  Branches      239      239              
==========================================
+ Hits         1772     1774       +2     
  Misses         97       97              
  Partials       34       34
Impacted Files Coverage Δ
cesium/features/lomb_scargle.py 100% <100%> (ø) :arrow_up:

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 c490794...4f48763. Read the comment docs.