-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
apply gains to autocorrs #436
Comments
I have taken a brief look. While I do think that it will probably be ok to do as you suggest, it will also require changes to the |
This slipped my mind - should I give it a shot @o-smirnov? |
Yes, please! I am buried in admin at the moment. You need to fix the kernels, take out the "precaution", and set |
@o-smirnov @bennahugo See #437. I have tried it out locally and nothing seems to burst into flame. |
That's my definition of a successful pipeline run! |
Also when I don't get arrested. |
@bennahugo has requested that we apply gains to autocorrelations when forming corrected data.
At the moment we zero them "as a precaution" here, but I see no reason not to flag them with FL.SKIPSOL instead (as we do with baseline length selections right now) to have them ignored yet corrected.
The text was updated successfully, but these errors were encountered: