Skip to content

Chromatic noise#8

Open
jeremy-baier wants to merge 5 commits into
bencebecsy:mainfrom
jeremy-baier:jgb-add-chrom-delay
Open

Chromatic noise#8
jeremy-baier wants to merge 5 commits into
bencebecsy:mainfrom
jeremy-baier:jgb-add-chrom-delay

Conversation

@jeremy-baier
Copy link
Copy Markdown

This PR adds support for injecting chromatic (in radio frequency) noise — including dm noise — into a pulsar’s residuals.

@jeremy-baier jeremy-baier changed the title Chromatic noise DRAFT: Chromatic noise Jun 8, 2024
@bencebecsy
Copy link
Copy Markdown
Owner

I think this looks great @jeremy-baier !
Have you tested this for consistency with either 1) comparing the injected residuals between this and libstempo.toasim.add_dm or 2) via injecting stuff with this and recovering with usual enterprise analysis?
I think it would be good to see either of those before merging this

@jeremy-baier
Copy link
Copy Markdown
Author

I have not had a chance to test it yet @bencebecsy .
I will try the enterprise recovery and get back to you !

@bencebecsy
Copy link
Copy Markdown
Owner

@jeremy-baier I see there's been some changes to this since you opened the PR. Did you managed to test a recovery from this or a comparison with libstempo?

@jeremy-baier
Copy link
Copy Markdown
Author

ooops. looks like I like this languish for a bit.
I can confirm now that this works.
image
I think the slight bias in dm recovery is from the fact that I did not inject a DM1,DM2, but recovered with a DM1,DM2

@jeremy-baier jeremy-baier changed the title DRAFT: Chromatic noise Chromatic noise May 1, 2026
@jeremy-baier
Copy link
Copy Markdown
Author

this could probably be merged now

@jeremy-baier
Copy link
Copy Markdown
Author

@bencebecsy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants