-
Notifications
You must be signed in to change notification settings - Fork 119
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
Possible issue in eq 36.115 #275
Comments
I'm not sure. I'll ask @vveitch |
Yes, this is a typo. (There should be a hat{p}(a) in the estimator in the same position as p(a) in the estimand). I'll fix it tonight. |
Hi @vveitch .I'm currently doing a pass of bug fixes, I will check in the source latex by EOD, please ping me before pulling :) |
in that case, I'll just do tomorrow morning :)
…On Mon, Aug 14, 2023 at 2:49 PM Kevin P Murphy ***@***.***> wrote:
Hi @vveitch <https://github.com/vveitch> .I'm currently doing a pass of
bug fixes, I will check in the source latex by EOD, please ping me before
pulling :)
—
Reply to this email directly, view it on GitHub
<#275 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABZZFQWPWHN3EOKSPD773DDXVJ6K3ANCNFSM6AAAAAA22JBZCQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I just committed all the changes. Please check your email for some more context :) |
fixed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Edition: 2023-04-01, Advanced Topics
I could merely be confused, but it seems there is a mismatch between the equations in red:
Shouldn't p(a) appear in 36.115? It seems like we might be trying to accomplish that with 1/|A|, but I don't believe 1/|A| = p(a)
The text was updated successfully, but these errors were encountered: