-
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
Make whitened residuals available #1671
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1671 +/- ##
=======================================
Coverage 68.47% 68.47%
=======================================
Files 104 104
Lines 24289 24294 +5
Branches 4336 4336
=======================================
+ Hits 16631 16636 +5
Misses 6572 6572
Partials 1086 1086
☔ View full report in Codecov by Sentry. |
Code:
|
Does this work for WB data? |
@@ -500,6 +500,12 @@ def calc_time_resids( | |||
) | |||
return (phase_resids / self.get_PSR_freq(calctype=calctype)).to(u.s) | |||
|
|||
def calc_whitened_resids(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it better to have a separate method, or calc_time_resids()
with a flag?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be a separate method since whitened residuals are dimensionless, whereas time resids have the dimentions of time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then probably add a "See Also" to the other residual methods
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated documentation.
It should. But I haven't tried it yet. |
Looks good to me. ready to go? |
Yes. |
This partially overlaps with the stale PR #1000.
whitened_resids = (resids - noise_resids) / scaled_toa_error
If the fitting has been done properly, the whitened residuals should be normally distributed with unit variance.