-
Notifications
You must be signed in to change notification settings - Fork 8
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
Feature: add bootstrap CI to the rwa method #6
base: master
Are you sure you want to change the base?
Conversation
work in progress code - does not work yet
Hey Martin,
The code in there for review doesn't seem to match any of the material I
sent for implementing the bootstrap. How can I best help integrate that?
Best,
Scott T
…On Tue, Jun 29, 2021 at 6:16 AM Martin Chan ***@***.***> wrote:
@martinctc <https://github.com/martinctc> requested your review on: #6
<#6> Feature: add bootstrap CI to
the rwa method.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#6 (comment)>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIT53LWAU2G7XD5VX5BQBADTVGMOLANCNFSM4XDUG75A>
.
|
Hey @stonid this pull request is still a draft and I haven't properly coded anything yet! Just wanted to add you to this, will ping you when it is ready for review. |
Got it. I got a ping that I was supposed to review and was confused.
…On Wed, Jun 30, 2021, 4:02 PM Martin Chan ***@***.***> wrote:
Hey Martin,
The code in there for review doesn't seem to match any of the material I
sent for implementing the bootstrap. How can I best help integrate that?
Best,
Scott T
On Tue, Jun 29, 2021 at 6:16 AM Martin Chan *@*.***>
wrote:
@martinctc <https://github.com/martinctc> https://github.com/martinctc
requested your review on: #6 <#6>
#6 <#6> Feature: add bootstrap CI to
the rwa method.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
#6 (comment) <#6 (comment)>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIT53LWAU2G7XD5VX5BQBADTVGMOLANCNFSM4XDUG75A
.
Hey @stonid <https://github.com/stonid> this pull request is still a
draft and I haven't properly coded anything yet! Just wanted to add you to
this, will ping you when it is ready for review.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIT53LRCXHVXSKZ2ETKINDDTVNZ6JANCNFSM4XDUG75A>
.
|
Summary
This branch adds the bootstrap method to extract confidence intervals for
rwa()
.[NOTE: THIS IS STILL A DRAFT - REMOVE THIS LINE WHEN PULL REQUEST IS READY FOR REVIEW]
Changes
The changes made in this PR are:
Check
(OPTIONAL) Note
This fixes #1.