Closed madslupe closed 4 years ago
Thomas, you think this is ready for merging?
Nice one!
Thanks! Should I update to 0.7.6 as well?
pypi? yes!
On Wed, Apr 15, 2020 at 3:56 PM madslupe notifications@github.com wrote:
Thanks! Should I update to 0.7.6 as well?
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/hddm-devs/hddm/pull/58#issuecomment-614055993, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFETGFX27S7VBWIOLCB7LTRMW4KDANCNFSM4LSWN32A .
Created a new class called HDDMrlRegressor that allows estimating instrumental learning with the RLDDM while also estimating the impact of continuous regressors. It builds on the HDDMRegressor, but adds a learning rate parameter and takes as input feedback on choices. A potential issue is that I also included alpha as a parameter for the normal HDDMRegressor, because I create alpha in base.py. Ideally this would be created within the HDDMrlRegressor class, but I didn't get that working when also letting regressors impact alpha.