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
Hi all ! And many thanks for your time on MAPIE development, this library rocks !
I found a syntax discrepancy between CQR and other regression classes, that makes using the library a little bit awkward when performing benchmarks.
For example, the MapieRegressor and TimeSeriesRegressor classes both have an attribute called single_estimator_, that is clear from the docstring and represents the model we would use without MAPIE.
However, the MapieQuantileRegressor class has no such attributes (which is surprising for a class inheriting MapieRegressor which does have this attribute), but instead the model that would be naturally used without MAPIE, the 0.5-quantile estimator, is hidden in estimators_[2].
It thus happens that in a code comparing MapieRegressor with MapieQuantileRegressor, the code syntax is not homogeneous.
Would it be possible to make a single_estimator_ attribute for MapieQuantileRegressor in the same spirit as MapieRegressor ?
The text was updated successfully, but these errors were encountered:
Hi all ! And many thanks for your time on MAPIE development, this library rocks !
I found a syntax discrepancy between CQR and other regression classes, that makes using the library a little bit awkward when performing benchmarks.
For example, the
MapieRegressor
andTimeSeriesRegressor
classes both have an attribute calledsingle_estimator_
, that is clear from the docstring and represents the model we would use without MAPIE.However, the
MapieQuantileRegressor
class has no such attributes (which is surprising for a class inheritingMapieRegressor
which does have this attribute), but instead the model that would be naturally used without MAPIE, the 0.5-quantile estimator, is hidden inestimators_[2]
.It thus happens that in a code comparing
MapieRegressor
withMapieQuantileRegressor
, the code syntax is not homogeneous.Would it be possible to make a
single_estimator_
attribute forMapieQuantileRegressor
in the same spirit asMapieRegressor
?The text was updated successfully, but these errors were encountered: