Stanislav Khrapov
Stanislav Khrapov
I think this is somewhat related to https://github.com/alan-turing-institute/sktime/pull/3075 and the underlying issue https://github.com/alan-turing-institute/sktime/issues/3071. The splitter's test window is defined by the values in the forecasting horizon, without implicit modifications. Assuming...
@ngupta23 Exactly my point. I do not think the blame is on splitters themselves. I understand that, despite the explicit wish of a user for a non-full range forecasting horizon,...
@ngupta23 Just to understand your proposal very clearly. You want `sktime.forecasting.model_selection._split.BaseSplitter.split` to return a tuple of 4 arrays, instead of 2? `y_train, y_test, X_train, X_test`? And it is expected that...
@ngupta23 I am very sorry to make you wait so much. I have very little time lately for my hobby activities like this one :( I've been scratching my head...
Btw, who is paying for all this computational time on GitHub?.. It's not just about developer experience, is it?
Dear @mwahal. Sorry for the late response. I have just run the code. It works perfectly well. But, I use Python 3.4. You are on Python 2.7. Unfortunately, it is...
Are you using exactly the data that you printed in your last comment? If so, then these two data sets do not have common dates and the merge will produce...
Again, my apologies. The data (csv files) were excluded from git commits. I pushed the data to the repo just now. Sorry...
@mwahal Thanks for raising all these issues! Although I have plenty of data to test this code on, I never did it yet. Please, see the last two commits. They...
Dear all, it would be totally awesome to release this. The issue exists several months and several versions already https://github.com/tiangolo/sqlmodel/discussions/725. Please, let's get some traction here :)