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

Temporal leaks #119

Open
ThomasBourgeois opened this issue Dec 8, 2020 · 2 comments
Open

Temporal leaks #119

ThomasBourgeois opened this issue Dec 8, 2020 · 2 comments

Comments

@ThomasBourgeois
Copy link

Hi, very nice lib Axel,

I just tried it, is there any way to deal with temporal leaks (eg for regression).

Currently I'm unsure the lib deals with temporeal leaks, for example the current score that is spit out of Optimizer is a cross validation score but that probably is slightlty overestimated in case some folds are trained in the future compared to the validation set.

-> one way to deal with this is currently to do time-series cross validation with the following method:
https://robjhyndman.com/hyndsight/tscv/

Sklearn has implemented it with the folowing : https://scikit-learn.org/stable/modules/generated/sklearn.model_selection.TimeSeriesSplit.html

that could be a good PR for mlbox !

Best,
Thomas

@AxeldeRomblay
Copy link
Owner

AxeldeRomblay commented Dec 23, 2020

Hello @ThomasBourgeois,
Thank you !
Indeed MLBox uses standard cv, but the problem with custom cv is that it is not automatic and/or increases the number of parameters in the Optimiser class... Nevertheless it should be easy to tweak the code here to add your own cv : https://github.com/AxeldeRomblay/MLBox/blob/master/mlbox/optimisation/optimiser.py#L424

See #83 for more details ;)
Axel

@ThomasBourgeois
Copy link
Author

ThomasBourgeois commented Dec 27, 2020

Hi @AxeldeRomblay , thx for your reply, I hadnt seen this was already referenced. Wow, tweaking the code with throw-away lines, that is a no-go for me, but if you think there is no other way, no pb. ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants