mljar-supervised icon indicating copy to clipboard operation
mljar-supervised copied to clipboard
verified publisher icon mljar

A little 🐛 in the printing of XGBoost results

Open munozariasjm opened this issue 3 years ago • 6 comments

Hi!,

Just to report a little (and unimportant) bug. When the results of the XGBoost models are printed, I heavily suspect (😂) the rsme and the fitting time are being interchanged: 6_Xgboost rmse 11.868675 trained in 1264.67 seconds Clearly it should be: 6_Xgboost rmse 1264.67 trained in 11.868675 seconds If necessary, tell me, I can open a PR with the fix. Regards and keep the excellent work on!!!

munozariasjm avatar Apr 08 '22 12:04 munozariasjm

Hi @munozariasjm,

It should be good. Please check the code: https://github.com/mljar/mljar-supervised/blob/04a90ffbff33b2c93a7c212825b987e73b7f62fe/supervised/base_automl.py#L290-L294

pplonski avatar Apr 08 '22 12:04 pplonski

Why do you think that they are interchanged?

pplonski avatar Apr 08 '22 12:04 pplonski

There is no bug, I'm sure. munozariasjm doesn't know what he's talking about, it seems.

I'd closed the "issue".

unreal79 avatar Apr 18 '22 10:04 unreal79

Ok, @unreal79. Let's suppose there is absolutely no bug... Then please explain to me how come we can print the RMSE for this toy data obtained by the linear model is: 3_Linear rmse 0.0 trained in 13.17 seconds Here is the notebook.

I still don't know where these printing bugs are coming from, but it is not normal.

munozariasjm avatar Apr 18 '22 14:04 munozariasjm

Thank you @munozariasjm for example code to reproduce the issue. It might be a bug with get_final_loss() method.

Is anyone interested in tracking the bug?

pplonski avatar Apr 19 '22 07:04 pplonski

@pplonski Not able to understand the error at first place ,can you please elaborate so i can start working on it?

Vamp1899 avatar Jul 19 '22 04:07 Vamp1899