-
Notifications
You must be signed in to change notification settings - Fork 1
Picking Simulations #93
Comments
I agree that we should throw the beta tables out. That leaves these as characteristics that we can vary (along with the purpose that they serve in the paper):
|
So basically the sim section should go like:
Does that seem like the plan? Maybe I should swap 3 and 4? |
That would work. Though I would say you maybe don't need 3. Especially if we can add like a quick theoretical note on the exponential transform and/or curse of too many instruments |
Okay, let me write that up now. |
I just pushed the changes. I ended up putting just the p-charts in the appendix (for every N value and for both exp and no exp). Could someone read through my section to make sure it all seems reasonable? |
I am reading and editing now
…On Fri, Jun 4, 2021 at 4:18 PM marionoro ***@***.***> wrote:
I just pushed the changes. I ended up putting just the p-charts in the
appendix (for every N value and for both exp and no exp). Could someone
read through my section to make sure it all seems reasonable?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#93 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQCGE4O4ZR6NJ5PUIRYHXIDTREYLBANCNFSM46CXNA2Q>
.
|
So you didn't think the rho charts added anything really? I guess that seems plausible. I know that like the beta charts didn't show much at all of course. And probably no need to mention the me_cov stuff. |
Ok yeah the rho chart we have in the main text is pretty good. |
Let me read over that new paragraph, in the mean time I guess you should proofread everything else |
Alright, made some revisions. But overall I think this issue is closed. I feel pretty good about the sim section, just need to work on everything else now. |
Here's what I suggest. Pick two tables to put in the main text:
I don't think the beta tables show much so maybe we should completely toss those out.
The text was updated successfully, but these errors were encountered: