If one sets the wrong estimator name, UnboundLocalError is returned. The variable effects is returned before being assigned. A better approach would be to define effects = [np.nan]*5 at the start of the function. It would avoid repetition. Also, we should produce a Warning in the case of a wrong estimator name.
In get_estimation.py, get_estimation() :
If one sets the wrong estimator name, UnboundLocalError is returned. The variable
effects
is returned before being assigned. A better approach would be to defineeffects = [np.nan]*5
at the start of the function. It would avoid repetition. Also, we should produce a Warning in the case of a wrong estimator name.