stingray icon indicating copy to clipboard operation
stingray copied to clipboard

Streaming Timeseries

Open matteobachetti opened this issue 1 year ago • 2 comments

These new methods serve the purpose of streaming time series objects by segment. This is a prerequisite for the parallelization of the Averaged*spectrum calculation.

matteobachetti avatar Sep 04 '24 15:09 matteobachetti

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

Line 1472:69: E203 whitespace before ':' Line 1476:41: E203 whitespace before ':' Line 2802:101: E501 line too long (111 > 100 characters)

Comment last updated at 2024-09-09 08:58:27 UTC

pep8speaks avatar Sep 04 '24 15:09 pep8speaks

Codecov Report

Attention: Patch coverage is 95.20000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 79.20%. Comparing base (241f81a) to head (0e0c1c8).

Files with missing lines Patch % Lines
stingray/base.py 91.52% 5 Missing :warning:
stingray/gti.py 98.41% 1 Missing :warning:

:exclamation: There is a different number of reports uploaded between BASE (241f81a) and HEAD (0e0c1c8). Click for more details.

HEAD has 74 uploads less than BASE
Flag BASE (241f81a) HEAD (0e0c1c8)
78 4
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #838       +/-   ##
===========================================
- Coverage   96.53%   79.20%   -17.33%     
===========================================
  Files          48       48               
  Lines        9257     9336       +79     
===========================================
- Hits         8936     7395     -1541     
- Misses        321     1941     +1620     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Sep 05 '24 14:09 codecov[bot]

I converted to draft. There is some work to be done

matteobachetti avatar Nov 11 '24 09:11 matteobachetti