-
Notifications
You must be signed in to change notification settings - Fork 2
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
Scaled and Shifted #16
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## release/v0.19.0 #16 +/- ##
===================================================
- Coverage 81.49% 80.82% -0.68%
===================================================
Files 93 97 +4
Lines 18810 19241 +431
Branches 18810 19241 +431
===================================================
+ Hits 15329 15551 +222
- Misses 3382 3591 +209
Partials 99 99 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Some things to confirm, if you could:
I checked these but it's manual so a second look would help 🙏 |
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 have a few questions, but overall, this looks great! I love all the tests!
Generally the idea is
Shiftable
traitGaussian
,some_dist.shifted(dx)
gets absorbed and just gives another instance of the distributionShifted
ShiftedSuffStat
that shifts the data before feeding into the suffstat of the parentIt will be similar for
Scaled