-
Notifications
You must be signed in to change notification settings - Fork 782
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
Why I got different lift result when using get_cumlift() and calculating line by line? #706
Comments
hi @AmyLin0515 A couple ideas: See the code for causalml/causalml/metrics/visualize.py Lines 54 to 135 in c154afe
causalml/causalml/metrics/visualize.py Lines 90 to 93 in c154afe
causalml/causalml/metrics/visualize.py Lines 102 to 104 in c154afe
Not sure if you are providing the If you're not providing FYI, also see work in #707 |
Hi @ras44 ! Thanks for providing insights. I did find the difference decreased a lot after I added 10 random columns and included them to sort. However, I don't understand why we need to add these two random columns. And if eventually the order was changed by the final 10th random columns, what is the point that we added so many of them. |
Hi @AmyLin0515. We updated the uplift/Qini curve and score calculation to use the theoretical random instead of the sampled random in #799. With the change, we don't use a set of ten random estimates to calculate the AUUC and Qini scores for random, which will always be 0.5 and 0.0, respectively. To investigate the discrepancy between the manual calculation vs. the |
Describe the bug
Hi Team!
I used get_cumlift(), and got the lift for S-Learner like this:
When I tried to duplicate the result, calculating it manually, the result is different from what I had using get_cumlift().
This is how table looks like:
And then I calculate the lift:
This is how the final result looks like:
I plot the difference between the result from get_cumlif() and manual calculation.
Does anyone know why they are different?
Environment (please complete the following information):
pandas
,scikit-learn
,cython
):pandas==1.3.5
,scikit-learn==1.0.2
,cython==0.29.34
]The text was updated successfully, but these errors were encountered: