p_drop / n_drop in bandits/ccb estimators

VowpalWabbit / estimators

Estimators to perform off-policy evaluation

BSD 3-Clause "New" or "Revised" License

13 stars 17 forks source link

p_drop / n_drop in bandits/ccb estimators #66

Closed ataymano closed 2 years ago

ataymano commented 2 years ago

1) count is removed since semantics of interaction between count and n_drop is confusing (whether n_drop or count * n_drop events are actually dropped) + usage in real scenarios is limited 2) p_drop / n_drop support is added

ataymano commented 2 years ago

@pmineiro , can you take a look?