Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing random state in Quantile Regression #405

Open
gmartinonQM opened this issue Jan 25, 2024 · 1 comment · May be fixed by #424
Open

Missing random state in Quantile Regression #405

gmartinonQM opened this issue Jan 25, 2024 · 1 comment · May be fixed by #424
Assignees
Labels
Bug Type: bug Good first issue Easy issue to start to contribute to MAPIE Needs decision The MAPIE team is deciding what to do next. Regression Related to regression (excluding time series) Source: developers Proposed by developers.

Comments

@gmartinonQM
Copy link
Collaborator

I am trying to get reproducible results with MapieQuantileRegressor with the "split" method. Unfortunately :

TypeError: MapieQuantileRegressor.__init__() got an unexpected keyword argument 'random_state'

Whereas the attribute random_state is indeed used in the code of MapieQuantileRegressor, but always with the default value None. (

random_state=random_state,
)

Could you please add the random_state in the constructor of MapieQuantileRegressor ?

@gmartinonQM gmartinonQM added the Enhancement Type: enhancement (new feature or request) label Jan 25, 2024
@LacombeLouis
Copy link
Collaborator

Hi @gmartinonQM, thank you for this information.
If you want replicable results for the moment, I would suggest using the random_state in the .fit method. However, this is indeed something we should change to have it in the .__init__() of the method.
Thank you!

@LacombeLouis LacombeLouis added Bug Type: bug Good first issue Easy issue to start to contribute to MAPIE and removed Enhancement Type: enhancement (new feature or request) labels Jan 26, 2024
@thibaultcordier thibaultcordier added the Source: developers Proposed by developers. label Mar 1, 2024
@tiagoleonmelo tiagoleonmelo linked a pull request Mar 22, 2024 that will close this issue
8 tasks
@jawadhussein462 jawadhussein462 added MAPIE v1 Indicate that the issue will be resolved with the release of MAPIE v1 Regression Related to regression (excluding time series) Discussion in progress Discussion ongoing between the Mapie team and the author. Needs decision The MAPIE team is deciding what to do next. and removed MAPIE v1 Indicate that the issue will be resolved with the release of MAPIE v1 Discussion in progress Discussion ongoing between the Mapie team and the author. labels Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Type: bug Good first issue Easy issue to start to contribute to MAPIE Needs decision The MAPIE team is deciding what to do next. Regression Related to regression (excluding time series) Source: developers Proposed by developers.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants