-
Notifications
You must be signed in to change notification settings - Fork 102
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
Fourier series representation of chromatic noise (CMWaveX
)
#1802
Conversation
CMWaveX
)CMWaveX
)
Here is a comparison between PINT and ENTERPRISE analyses for chromatic noise to ensure the conventions are consistent. This uses a simulated dataset. This is similar to the simulation studies presented in https://arxiv.org/abs/2405.01977 Parameter estimation |
CMWaveX
)CMWaveX
)
It looks like almost all of the changes to |
This is nice. Since it didn't make it into the paper, and this PR will be hard to find once it is merged, will these results be preserved somehow? Maybe a notebook (but not one that is automatically run unless it is fast)? |
Only the changes to |
OK, I think the code looks fine here. Any thoughts on the docs? |
I will add an example notebook as you suggested. |
Example added to notebook. |
So is that notebook one that is built & run as part of CI? If so, any estimate of the additional run-time? |
~10 s on my laptop. |
OK, good. I can merge when ready then. |
Ok... please do.
…On Tue, 27 Aug, 2024, 4:37 pm David Kaplan, ***@***.***> wrote:
OK, good. I can merge when ready then.
—
Reply to this email directly, view it on GitHub
<#1802 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE6CGN3WGSYWFUIBVDUTWELZTSFLFAVCNFSM6AAAAABKWQLAXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJSG42TEMJVGM>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***>
|
See #1661.
Like
WaveX
&DMWaveX
#1797 needs to be merged first. I have merged that PR into this one to implement some validation steps that depend on both
CMWaveX
andPLChromNoise
.