-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Efficient influence function constructed from influence_fn
errors if called multiple times
#483
Comments
@agrawalraj , this is now somewhat blocking #484 , as (our version of) TMLE involves evaluating the influence function on |
I can't reproduce this on my machine with |
I'm using |
I believe that is consistent with the (implied) requirements of ChiRho via Pyro. |
You're right, but I think the easiest short-term fix for this would just be to add a |
When I upgrade torch to |
It looks like this error doesn't appear in the CI builds even with |
chirho.robust.ops.influence_fn
at https://github.com/BasisResearch/chirho/blob/staging-robust/chirho/robust/ops.py#L18 return a function for approximating the efficient influence function at a collection of data points. If the returned function is called multiple times it results in the following (somewhat opaque) error:For example, this error can be seen in the current tests by adding multiple calls to the generated function as follows:
https://github.com/BasisResearch/chirho/blob/sw-multiple-eif/tests/robust/test_ops.py#L80
The text was updated successfully, but these errors were encountered: