Geert Barentsen
Geert Barentsen
@danhey Thanks for the feedback! I didn't pay attention to maintaining the existing defaults. I've changed it so you now get `Amplitude [normalized]` and `Period [d]` as the default plot....
Thank you for the feedback both! This all seems sensible. I'm hoping most of it can be resolved using small changes, but it may take me some time to get...
Thanks @danhey! I like the behavior demonstrated in the `gist` you posted. I do like the simplicity of just having `power` as a property, where the word "power" is used...
@ojhall94 @keatonb Do you want to give this branch another test? The `normalization` keyword is back and the `pg.amplitude` and `pg.psd` properties are gone, thanks to @danhey.
Thank you all for the thoughtful comments! Would it be correct to say that most concerns could be addressed if we made the following changes: 1) Make sure the light...
ping @danhey :wave:
@danhey That sounds like it may be an effective solution. Please ping me if you need help!
The flood of warnings is annoying, so I think we should suppress them! (which is easy) It does raise the valid question how `stitch()` should handle meta data from multiple...
For simplicity, I wonder if we should turn *anything* contained in `meta` into a list after `stitch()`. It means that you can't assume that e.g. `meta['teff']` is a float rather...
@bmorris3 I'm sorry to hear you are experiencing those intermittent failures. That sounds annoying! It would be good to understand exactly which part of the CBV correction is failing intermittently....