-
Notifications
You must be signed in to change notification settings - Fork 763
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
Add multi-armed Qini (maq) #662
Comments
hi @t-tte FYI, looks like the MAQ python package is available now: https://github.com/grf-labs/maq/tree/master/python-package with example: https://github.com/grf-labs/maq/blob/master/python-package/notebooks/introduction.ipynb |
Yep, @erikcs has now now polished the Python implementation. The idea we discussed was to implement multi-qini through CausalML.metrics.multiqini or something similar. Should be doable via a lightweight wrapper. Does this seem reasonable @erikcs, @ras44 and @jeongyoonlee? I'm currently short on time but can look into this early next year. Alternatively, if any of you have bandwidth for an initial implementation, I'll be happy to review. |
Thanks @t-tte, that sounds great! I too have a bit limited bandwidth now, but would be happy to provide input (and could also gladly hop on a zoom if anyone wants a quick live tour of how the multi-armed qini proposal works). And nice to see you @ras44, I remember you made some nice contributions to grf! |
I'm closing this issue as we merged #729. |
Is your feature request related to a problem? Please describe.
Current methods for evaluating multiple costly treatments are somewhat hacky.
Describe the solution you'd like
The good folks at grf labs came up with a generalisation of the Qini curve method for multiple costly treatments.
Describe alternatives you've considered
N/A
Additional context
@erikcs is currently working to polish the Python wrapper for maq. The idea is to incorporate this in a lightweight way to Causal ML.
The text was updated successfully, but these errors were encountered: