stingray icon indicating copy to clipboard operation
stingray copied to clipboard

Bayesian Excess Variance (Bexvar) in Stingray - GSoC'22 project

Open mihirtripathi97 opened this issue 3 years ago • 2 comments

This is the base code for implementing Bayesian Excess Variance (bexvar) in Stingray (#582).

This project is part of Google Summer of Code 2022, a brief summary of the project can be found here.

mihirtripathi97 avatar Jul 10 '22 17:07 mihirtripathi97

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

Line 55:20: E261 at least two spaces before inline comment Line 57:10: E261 at least two spaces before inline comment Line 75:90: E502 the backslash is redundant between brackets Line 76:9: E128 continuation line under-indented for visual indent Line 112:48: E203 whitespace before ',' Line 112:66: E502 the backslash is redundant between brackets Line 113:13: E128 continuation line under-indented for visual indent Line 118:1: E302 expected 2 blank lines, found 1 Line 148:17: E221 multiple spaces before operator Line 159:71: E502 the backslash is redundant between brackets Line 160:9: E128 continuation line under-indented for visual indent Line 168:49: E251 unexpected spaces around keyword / parameter equals Line 168:51: E251 unexpected spaces around keyword / parameter equals Line 168:66: E251 unexpected spaces around keyword / parameter equals Line 168:68: E251 unexpected spaces around keyword / parameter equals Line 168:83: E251 unexpected spaces around keyword / parameter equals Line 168:85: E251 unexpected spaces around keyword / parameter equals Line 184:101: E501 line too long (102 > 100 characters) Line 187:1: W293 blank line contains whitespace Line 200:79: W291 trailing whitespace Line 214:44: E203 whitespace before ',' Line 251:91: W291 trailing whitespace Line 265:56: E231 missing whitespace after ',' Line 268:48: E231 missing whitespace after ',' Line 269:44: E231 missing whitespace after ',' Line 271:43: E231 missing whitespace after ','

Line 13:1: E302 expected 2 blank lines, found 1

Comment last updated at 2022-07-22 12:25:47 UTC

pep8speaks avatar Jul 10 '22 17:07 pep8speaks

Codecov Report

Merging #664 (cad13a3) into main (1bd574a) will decrease coverage by 7.34%. The diff coverage is 45.55%.

:exclamation: Current head cad13a3 differs from pull request most recent head 9f48fbb. Consider uploading reports for the commit 9f48fbb to get more accurate results

@@            Coverage Diff             @@
##             main     #664      +/-   ##
==========================================
- Coverage   97.12%   89.78%   -7.35%     
==========================================
  Files          41       42       +1     
  Lines        7554     7644      +90     
==========================================
- Hits         7337     6863     -474     
- Misses        217      781     +564     
Impacted Files Coverage Δ
stingray/bexvar.py 45.55% <45.55%> (ø)
stingray/largememory.py 10.99% <0.00%> (-84.05%) :arrow_down:
stingray/pulse/overlapandsave/test_ols.py 68.62% <0.00%> (-31.38%) :arrow_down:
stingray/pulse/__init__.py 76.92% <0.00%> (-23.08%) :arrow_down:
stingray/base.py 77.15% <0.00%> (-19.83%) :arrow_down:
stingray/modeling/parameterestimation.py 67.29% <0.00%> (-18.75%) :arrow_down:
stingray/pulse/pulsar.py 89.43% <0.00%> (-9.16%) :arrow_down:
stingray/gti.py 94.91% <0.00%> (-4.33%) :arrow_down:
stingray/utils.py 95.19% <0.00%> (-3.81%) :arrow_down:
stingray/powerspectrum.py 95.65% <0.00%> (-3.77%) :arrow_down:
... and 6 more

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

codecov[bot] avatar Jul 20 '22 10:07 codecov[bot]