py-why / py-why/EconML

How to use known treatment probabilities in doubly robust learners

Open
#730 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Jupyter Notebook
Stars
4.8k
Forks
827
PR merge metrics
No merged PRs in 30d

Description

@kbattocchi

Hi Keith,

How would you recommend handling a case where we know the true treatment probabilities? I'd prefer to use them to avoid having to fit the model_propensity (in a doubly robust model, say ForestDRLearner).

A few options:

  1. Pass the (inverse) probabilities as sample_weight to fit. But then we need to choose something for model_propensity, perhaps just a dummy classifier?
  2. Create a trivial model_propensity that takes the probability as a feature and returns the same probability. But then we need some workaround to prevent the model_regression from using the probability as a feature (by DRLearner will always pass X, W to both model_propensity and model_regression). Maybe we can use a sklearn pipeline with transformer for this.

Thanks!
Kyle

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Review the ForestDRLearner and DRLearner fit entry points, focusing on model_propensity, model_regression, X, W, and sample_weight. Compare the proposed known-probability approaches and define behavior and tests for supplying probabilities without fitting model_propensity or exposing them to model_regression. No files or tests are named, so repository exploration and maintainer clarification are needed.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, scikit-learn
Domain
machine-learning
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.