StingraySoftware / stingray

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

Add find_bexvar() method in Lightcurve class and its relevant tests - GSoC"22 #669

Closed mihirtripathi97 closed 2 years ago

mihirtripathi97 commented 2 years ago

Added bexvar() method in Lightcurve class and its relevant tests as a part of GSoC"22 project.

pep8speaks commented 2 years ago

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

Line 85:99: W291 trailing whitespace Line 89:60: W291 trailing whitespace Line 256:101: E501 line too long (108 > 100 characters) Line 1803:101: E501 line too long (111 > 100 characters)

Line 793:26: E127 continuation line over-indented for visual indent Line 933:26: E127 continuation line over-indented for visual indent Line 963:26: E127 continuation line over-indented for visual indent Line 1133:101: E501 line too long (106 > 100 characters) Line 1421:1: E302 expected 2 blank lines, found 1 Line 1444:28: E127 continuation line over-indented for visual indent Line 1464:26: E127 continuation line over-indented for visual indent Line 1465:26: E127 continuation line over-indented for visual indent Line 1481:1: E302 expected 2 blank lines, found 1 Line 1508:26: E127 continuation line over-indented for visual indent Line 1510:94: W291 trailing whitespace Line 1514:26: E127 continuation line over-indented for visual indent Line 1521:1: W293 blank line contains whitespace Line 1522:101: E501 line too long (140 > 100 characters) Line 1530:88: W291 trailing whitespace Line 1531:30: E127 continuation line over-indented for visual indent Line 1532:30: E127 continuation line over-indented for visual indent Line 1565:34: E231 missing whitespace after ',' Line 1602:30: E127 continuation line over-indented for visual indent Line 1639:101: E501 line too long (103 > 100 characters) Line 1679:101: E501 line too long (103 > 100 characters)

Comment last updated at 2022-09-12 07:01:25 UTC
codecov[bot] commented 2 years ago

Codecov Report

Merging #669 (a499c5b) into main (f7c981d) will increase coverage by 0.02%. The diff coverage is 98.34%.

@@            Coverage Diff             @@
##             main     #669      +/-   ##
==========================================
+ Coverage   97.14%   97.17%   +0.02%     
==========================================
  Files          42       42              
  Lines        7644     7713      +69     
==========================================
+ Hits         7426     7495      +69     
  Misses        218      218              
Impacted Files Coverage Δ
stingray/lightcurve.py 97.95% <97.93%> (+0.17%) :arrow_up:
stingray/gti.py 99.27% <100.00%> (+0.03%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more