-
Notifications
You must be signed in to change notification settings - Fork 101
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
Bayesian interface for Wideband data (No correlated noise) #1426
Conversation
The corner plot looks good. Can you add the best-fit values from standard fitting to the plot (as truths)? Or at least confirm that it agrees? |
Do you mean the pre-fit values present in the par file or post-fit values obtained using something like |
post-fit |
I think the docstring formatting is slightly different than what we've adopted elsewhere. For instance, it should be:
so the data-type and description should go on different lines. This will be easiest to see when the RTD build finishes, or if you build locally, but it's not doing all of the normal parsing as-is. (Not all of the indenting is right above, but hopefully you get the idea) |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #1426 +/- ##
=======================================
Coverage 68.54% 68.55%
=======================================
Files 103 103
Lines 23502 23514 +12
Branches 4098 4098
=======================================
+ Hits 16110 16120 +10
- Misses 6377 6378 +1
- Partials 1015 1016 +1
☔ View full report in Codecov by Sentry. |
The RTD build failed, but that may have just been a timeout. Otherwise this looks good but I agree the fit discrepancy should be sorted out. |
New notebooks look good. |
This is ready to go, then. |
I have implemented the Bayesian interface for analyzing wideband data. Correlated noise is still TODO..
I have created a new example script
bayesian-wideband-example.py
for this feature.Here is a posterior corner plot from running this script.