Amit Sharma

Z represents an instrumental variable, as shown by the output of the `identify_effect` method. But it is not observed in the data, so we do not use it.

thanks @lgautier for this contribution. I like the idea. Added a few comments above.

@lgautier Checking in to see if we can merge this. Can you have a look at the CLA above and sign if everything's okay?

Thank you for raising this @kangqiao-ctrl. Yeah, you are right about the `qcut` functionality. It should not be happening for binary variables. Let me try to add a fix and...

thank you for raising this issue @kenleejr, and for looking deeper to identify the source of this dimension mismatch. I will have a look and raise a PR to fix...

That is a good idea @adg-ci. Any thoughts on how we can achieve that in the cleanest manner (i.e., without having to parallelize each refuter separately)? If you have any...

It may be a good idea to start with parallelization when looping through a list of refutation methods. This can be done without changing any internal code: we simply need...

thanks for the code example, @astoeffelbauer . Let's go with joblib since it is fastest and EconML also uses it., so that will lead to some consistency benefit. I'd suggest...

We do not support it currently. That's been on the desired features list for some time. Will add it for the December release.

It does use these arguments. They are passed to `__init__` method of the class. The docstring needs to be improved---the refute_estimate in the docstring refers to the user-facing `refute_estimate` from...