8000 add NE_SW parameter to the designmatrix by siyuan-chen · Pull Request #71 · vallis/libstempo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

add NE_SW parameter to the designmatrix #71

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

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

siyuan-chen
Copy link
Collaborator

This is the second PR related to adding tempo2 functionality to libstempo, so that the IPTA DR3 combination can be correctly analyzed with libstempo and enterprise.

The NE_SW parameter defining the average density of electrons from the spherical solar wind model is added to the designmatrix as a fully working parameter, that can be fixed (flag value 0), fitted (flag value 1) and marginalized for the data analysis.

E.g in the par file: NE_SW 7.9 0 or NE_SW 7.9 1

@siyuan-chen
Copy link
Collaborator Author

Hi @mattpitkin , this is the other PR merge into libstempo, so that the NE_SW parameter is properly taken into account. This PR should be fine as I just added parameters and variables that are passed through libstempo. If it looks fine to you, feel free to merge. I want to ask some of the testers of the other PR whether things work fine for them, before we merge that as well. Thanks for your help.

@mattpitkin
Copy link
Collaborator

Looks good to me. I'm happy to merge this in.

@mattpitkin mattpitkin merged commit 60c76f3 into vallis:master Aug 8, 2024
12 checks passed
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
0