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
Sanit checking of weighted results #18
Comments
Awesome, thanks! I wonder if we could check in your tests to prevent future
regressions? If it's not a self contained r script, no worries
…On Wed, Feb 9, 2022, 7:29 PM Robin ***@***.***> wrote:
Closed #18 <#18>.
—
Reply to this email directly, view it on GitHub
<#18 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAMWLF72TCJ4LU6U6PABC4DU2K6C5ANCNFSM5N6NZGNQ>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
I could do that but would take some time. Reopening to track progress, note: I may be able to get it to general principle stage and can provide more example input data if that would be useful? |
Not a priority at all. I should really find better techniques for unit testing in Rust; I'll think about doing that without depending on so many test data files checked in |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There are some statistical sanity checks in #7 but I've just done some more sanity checks and the results are good. Summary of them below. Three schools in one zone:
With weights, how many trips have destinations at each?
Strong near-linear positive relationship between n_trips and weight:
The text was updated successfully, but these errors were encountered: