Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.Sign up
Fix stream to return lst_mdl class instead of list #144
I was trying to follow your slides for the fasster package, but, as you noted in of the issues, the API has changed a bit. So I tried to adapt the code to make it work.
The essence of the problem: after applying stream() to the original fit, I get a model which is structured differently, causing forecast() on the updated package to fail.
Here's the code:
And after the stream() call:
Notice vs . Naturally, forecast.list is not found. After debugging for quite some time I fixed like in this PR. The forecast() function works as intended now.
I'm not sure if it's well aligned with other libraries which depend on fabletools. This fixes the issue for fasster though.
I can work on this further if it's an ok solution, like add tests.