Upload Causal IV case study in education folder #219
Conversation
| target += log_sum_exp( | ||
| log(1 - pi_c) + bernoulli_lpmf(Y[n] | eta_n), // Never-taker | ||
| log(pi_c) + bernoulli_lpmf(Y[n] | eta_c0)); // Complier |
There was a problem hiding this comment.
This is more numerically stable and efficient as:
target += log_mix(pi_c, bernoulli_lpmf(Y[n] | eta_c0), bernoulli_lpmf(Y[n] | eta_n));| // Superpopulation complier average causal effect (CACE) | ||
| // in per-1000 units | ||
| real CACE; | ||
| CACE = (eta_c1 - eta_c0)*10^3; |
There was a problem hiding this comment.
It doesn't look like you use this CACE value in your model. If it's just a derived quantity you'll be using in post-processing/analysis then you should move this to the generated quantities block
| @@ -0,0 +1,57 @@ | |||
| data { | |||
There was a problem hiding this comment.
My comments from the other Stan model also apply to this one
| if(Z[n] == 1 && W[n] == 1){ | ||
| target += log(pi_c) + bernoulli_lpmf(Y[n] | eta_c1) ; | ||
| } | ||
|
|
||
| // Never-taker (assigned to treatment) | ||
| else if(Z[n] == 1 && W[n] == 0){ | ||
| target += log(1 - pi_c) + bernoulli_lpmf(Y[n] | eta_n); | ||
| } |
There was a problem hiding this comment.
It's a bit more efficient to calculate log(pi_c) and log(1-pi_c) once and then reuse them:
model {
real log_pi_c = log(pi_c);
real log1m_pi_c = log1m(pi_c);
...
}
|
Hello @andrjohns, Thank you for your insights and guidance. I've updated the Stan code in accordance with your suggestions and removed the .DS_Store files. Please take a moment to review the modifications, and let me know if there are any additional adjustments needed prior to the publication of these files. |
The CI (model compilation) is still failing because your Stan models are using the old array syntax which has been removed from Stan. See this section of the manual for guidance on how to update |
|
I have updated all Stan models with the new array syntax. |
| transformed parameters { | ||
| // Superpopulation complier average causal effect (CACE) | ||
| // in per-1000 units | ||
| real CACE = (eta_c1 - eta_c0) * 10^3; |
There was a problem hiding this comment.
Move to generated quantities block
| real CACE = (eta_c1 - eta_c0) * 10^3; | ||
| real NACE = (eta_n1 - eta_n0) * 10^3; |
There was a problem hiding this comment.
Move to generated quantities block
|
I have made the necessary adjustments to the Stan code, transitioning the codes from the "transformed parameters" section to the "generated quantities" section. Additionally, I have removed all .DS_Store files from the repository. Could you proceed to merge this Pull Request? I am looking forward to the final sign-off. |
|
Thanks. Looks good. That label from GitHub that says they're -6kb is confusing, because they appear to have been deleted when I look at your branch. I can merge. |
Hello, I've uploaded a case study document titled "Instrumental Variables Analysis of Randomized Experiments with One-Sided Noncompliance." This study is a collaborative work with Profs. Avi Feller and Sophia Rabe-Hesketh from UC Berkeley. The document is a result of our work on the IES project, which you can review here: IES Project Link. I would greatly appreciate it if you could approve this merge request. Thank you.