CausalEstimator reporting a 90% instead of 95% confidence interval for bootstrapping?
YichenTang97 opened this issue · comments
Problem description
The current implementation of _estimate_confidence_intervals_with_bootstrap
method under the CausalEstimator
class might be reporting a 90% confidence interval (CI) instead of the 95% CI under the default confidence_level
setting of 0.95.
The current implementation for obtaining the CI seem to follow the Pivotal Intervals method (see section 8.3 of [1], and section 6 of the reading material refered in the code comment [2]). Given
where
For a significance level _estimate_confidence_intervals_with_bootstrap
method is returning confidence_level=0.95
, which in fact reports the 90% CI.
Could someone investigate into this and make changes if necessory? It would also be helpful to implement an option for choosing the computing method for CI (e.g. pivotal, percentile, normal, etc.).
Version information
- DoWhy v0.10.1
References
[1] L. Wasserman, All of statistics: a concise course in statistical inference, vol. 26. Springer, 2004.
[2] Reading 24 Bootstrap Confidence Intervals (https://ocw.mit.edu/courses/mathematics/18-05-introduction-to-probability-and-statistics-spring-2014/readings/MIT18_05S14_Reading24.pdf)
This issue is stale because it has been open for 14 days with no activity.
This issue was closed because it has been inactive for 7 days since being marked as stale.
thanks for raising this @YichenTang97 will take a look
This issue is stale because it has been open for 14 days with no activity.
This issue was closed because it has been inactive for 7 days since being marked as stale.