You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, contextual linear and logistic bandit algorithms share the same superclass BaseContextualPolicy.
The constructor of BaseContextualPolicy has alpha_ and lambda_ as arguments:
Length of a list of actions recommended in each impression.
When Open Bandit Dataset is used, 3 should be set.
batch_size: int, default=1
Number of samples used in a batch parameter update.
alpha_: float, default=1.
Prior parameter for the online logistic regression.
lambda_: float, default=1.
Regularization hyperparameter for the online logistic regression.
random_state: int, default=None
Controls the random seed in sampling actions.
"""
dim: int
n_actions: int
len_list: int=1
batch_size: int=1
alpha_: float=1.0
lambda_: float=1.0
random_state: Optional[int] =None
These arguments are used to initialize self.alpha_list and self.lambda_list, which are used by LogisticEpsilonGreedy, LogisticTS, and LogisticUCB but not used by LinearEpsilonGreedy, LinTS, and LinUCB.
I suggest moving alpha_, lambda_, self.alpha_list, and self.lambda_list to another class, BaseLogisticPolicy for example, and making logistic policies inherit this new class.
The text was updated successfully, but these errors were encountered:
Currently, contextual linear and logistic bandit algorithms share the same superclass
BaseContextualPolicy
.The constructor of
BaseContextualPolicy
has alpha_ and lambda_ as arguments:zr-obp/obp/policy/base.py
Lines 93 to 129 in c9ad20c
These arguments are used to initialize
self.alpha_list
andself.lambda_list
, which are used byLogisticEpsilonGreedy
,LogisticTS
, andLogisticUCB
but not used byLinearEpsilonGreedy
,LinTS
, andLinUCB
.I suggest moving
alpha_
,lambda_
,self.alpha_list
, andself.lambda_list
to another class,BaseLogisticPolicy
for example, and making logistic policies inherit this new class.The text was updated successfully, but these errors were encountered: