[BUG] LinearRegression get_estimator method's required arguments appear to do nothing about darts HOT 3 OPEN

Update: I am now seeing that it is due to this if statements. It's currently unclear to me what makes a model object subclass MultiOutputRegressor.

I still feel that logging.info is invisible to almost all users and having require args that are meaningless is misleading.

from darts.

Hi @a097123,

Darts relies on sklearn implementation for all the regression models. This MultiOuputRegressor class is implemented at this higher level, we do not have control over it but sometimes makes "single output" models supports multivariates series/ output_chunk_length > 1 series by wrapping them in this class.

In your code snippet, you implicitly use the sklearn's LinearRegression model which inherits from the MultiOutputMixin class. Because this model support multioutputs out of the box, following the information stated in the docstring, the model is directly returned (hence the unique ID).

If you look at model.model.coef_, you will see that there is one set of coefficient for each position in output_chunk_length (in accordance with your assumption since multi_models=True by default) but there is no straightforward way to access a specific estimator from the model.

We could move the sanity check one level in the method to avoid the unexpected behavior you reported.

Changing the logging message from info to warning is also a possibility, did not want to make this look too alarming but it seems to be counter-intuitive.

Note: Darts LinearRegressionModel class would be more appropriate if this is indeed the model you want to use since it supports some additional features such as probabilistic forecast.

from darts.

@madtoinou Thank you for the detailed response! This helps me understand what the class is doing much better. Also appreciate the suggestion of LinearRegressionModel and why it might be better.

I personally think that warning is better here but I am new to the lib and you might know better than me what a darts user might expect there. This might just be a weird edge case only a noob would hit before taking a more conventional approach.

from darts.