Daniel

Hey all, I think this is an important feature we should push ASAP! Right now, if I call ``` python pg = lc.to_periodogram(normalization='amplitude') pg.plot() ``` then the plot says the...

Hi Geert, awesome work! I can confirm that this works and I have played around with a few test systems. My main concern is that `pg.plot` now defaults to power...

I second @ojhall94's comment about keeping a `normalization` keyword. If I am completely honest, I think this PR makes the process of calculating a periodogram more complicated and less intuitive....

I've played around with @barentsen's changes and have implemented my thoughts. You can see it in action here: https://gist.github.com/danhey/53f1374ce87bf21cbaa5a8166292136c I feel strongly about keeping a `normalization` parameter to maintain default...

Thanks for taking a look @keatonb ! > Also, why not just call these `xunit` and `yunit` instead of `default_...`? "default" doesn't sound like something I should feel comfortable changing....

Hi all, sorry it's taking me so long to get around to this. I've pushed some very small fixes to the current implementation which should get it mostly working. At...

Hi all, I am having this exact same issue. When rpoly fails, the entire Python kernel crashes. This makes it difficult to do any sort of optimization over the light...

Hi Alex, Here is my YAML file :-) Let me know if you run into problems with it: ``` name: vespa channels: - conda-forge - defaults dependencies: - _libgcc_mutex=0.1=main -...

👍

See https://github.com/lightkurve/lightkurve-stats/pull/1