Periodogram() should have power normalisation options
See original GitHub issueProblem description
I’ve received some feedback from Prof. Tim Bedding that units of Power Spectral Density are useful for solar-type oscillators, but not necessarily for other types. Power Spectral Density should be the default, but other types could be possible.
I’ll seek out some more community feedback before building a functional example.
Example
pg = lightcurve.to_periodogram(normalization="psd")
Issue Analytics
- State:
- Created 5 years ago
- Reactions:1
- Comments:7 (5 by maintainers)
Top Results From Across the Web
Periodogram Normalization - CCRMA - Stanford University
Periodogram Normalization · To be true to its name, the integral (sum) over the Power Spectral Density (PSD) should equal the total signal...
Read more >Periodogram power spectral density estimate - MathWorks
This MATLAB function returns the periodogram power spectral density (PSD) estimate, pxx, of the input signal, x, found using a rectangular window.
Read more >Normalization of Power Spectral Density estimates
If X(t) is real the power spectrum S(f) is even; hence, we only need estimates for f ≥ 0. The single-sided. PSD is...
Read more >Lomb-Scargle Periodograms — Astropy v5.2
This form of the normalization ( normalization='standard' ) is the default choice used in LombScargle . The resulting power P is a dimensionless...
Read more >Understanding the Lomb–Scargle Periodogram - IOPscience
Abstract. The Lomb–Scargle periodogram is a well-known algorithm for detecting and characterizing periodic signals in unevenly sampled data.
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I struggle to think of anything more exciting than @danielhey preparing a PR and @ojhall94 testing and reviewing it. ❤️
Note: over in #403 we just tinkered with the API to provide a generic
Periodogram
parent class and specificLombScarglePeriodogam
andBoxLeastSquaresPeriodogram
sub classes. The latter hold extra data and methods which are specific to those methods. I imagine psd normalization can go in the parent class (@christinahedges ?), but either way you will want to make sure to start a PR from the most recent version of themaster
branch because of these very recent changes! (They were made to add support for BLS to lightkurve.)Hi Daniel,
If you wanted to do this one that would be great! You’ve got more experience on this topic so it would almost certainly turn out better than if I did it. Having it as a property is a neat idea, and a kwarg in from_lightcurve() was what I was thinking as well.
Cheers!