Skip to content
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

Merged
merged 36 commits into from
Aug 27, 2024

Conversation

abhisrkckl
Copy link
Contributor

@abhisrkckl abhisrkckl commented Jul 11, 2024

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 and PLChromNoise.

@abhisrkckl abhisrkckl changed the title Fourier series representation of chromatic noise (CMWaveX) WIP: Fourier series representation of chromatic noise (CMWaveX) Jul 11, 2024
@abhisrkckl
Copy link
Contributor Author

abhisrkckl commented Aug 7, 2024

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

Harmonic number selection
image

Parameter estimation
image
Red point is PINT estimate
Blue lines are injected values
Black corner plot is the ENTERPRISE posterior.

Chromatic noise spectrum
image

@abhisrkckl abhisrkckl changed the title WIP: Fourier series representation of chromatic noise (CMWaveX) Fourier series representation of chromatic noise (CMWaveX) Aug 7, 2024
@abhisrkckl abhisrkckl added the awaiting review This PR needs someone to review it so it can be merged label Aug 7, 2024
@dlakaplan
Copy link
Contributor

It looks like almost all of the changes to timing_model were general code updates not related to this PR. Is that right? I didn't examine them all in detail, but I'm guessing if all of the tests pass then they are probably OK.

@dlakaplan
Copy link
Contributor

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

Harmonic number selection image

Parameter estimation image Red point is PINT estimate Blue lines are injected values Black corner plot is the ENTERPRISE posterior.

Chromatic noise spectrum image

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)?

@abhisrkckl
Copy link
Contributor Author

It looks like almost all of the changes to timing_model were general code updates not related to this PR. Is that right? I didn't examine them all in detail, but I'm guessing if all of the tests pass then they are probably OK.

Only the changes to TimingModel.validate_component_types() method are related to this PR. Everything else is refactorings suggested by sourcery.

@abhisrkckl abhisrkckl closed this Aug 26, 2024
@abhisrkckl abhisrkckl reopened this Aug 26, 2024
@dlakaplan
Copy link
Contributor

OK, I think the code looks fine here. Any thoughts on the docs?

@abhisrkckl
Copy link
Contributor Author

I will add an example notebook as you suggested.

@abhisrkckl
Copy link
Contributor Author

OK, I think the code looks fine here. Any thoughts on the docs?

Example added to notebook.
https://nanograv-pint--1802.org.readthedocs.build/en/1802/examples/rednoise-fit-example.html#Chromatic-noise-fitting

@dlakaplan
Copy link
Contributor

So is that notebook one that is built & run as part of CI? If so, any estimate of the additional run-time?

@abhisrkckl
Copy link
Contributor Author

~10 s on my laptop.

@dlakaplan
Copy link
Contributor

OK, good. I can merge when ready then.

@abhisrkckl
Copy link
Contributor Author

abhisrkckl commented Aug 27, 2024 via email

@dlakaplan dlakaplan merged commit 916fcf8 into nanograv:master Aug 27, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review This PR needs someone to review it so it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants