TMLE update: include option of place clever covariate as the weights #173
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We have used
zepid
:TMLE
for a while and notice the estimate as well as estimate's variance under some setting are majorly different from R:tmle. (Due to company policy, I cannot share data with you).We think we are able to identify the root-cause of this discrepancy: R's implementation of
TMLE
use the "clever covariet" by weighting (target_gwt = True) in the latest release. After changingzepid
to have the same setting, we observe a more consistent alliance between R andzepid
. Here we want to share you our slight changes of codes and hope this can bring the function to be beneficial to a broader audience.Based on our empirical experience,
target_gwt = True
might be a more stable setting for general user.target_gwt = False
tends to give extreme result that is majorly different from IPW or AIPW. Whiletarget_gwt = True
gives more closer and stable result.target_gwt = True
will gives similar ATE astarget_gwt = False
with larger p-value. So the weighting approach might be more conservative and robust to weird data or model specification.Attached with some additional resources about this topics:
importance_sampling
):