search

StingraySoftware / stingray

Anything can happen in the next half hour (including spectral timing made easy)!
https://stingray.science/stingray
MIT License
174 stars 144 forks source link

Streaming Timeseries #838

Open matteobachetti opened 2 months ago

matteobachetti commented 2 months ago

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.

pep8speaks commented 2 months ago

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
codecov[bot] commented 2 months ago

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 ```diff @@ 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.

matteobachetti commented 1 week ago

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