-
Notifications
You must be signed in to change notification settings - Fork 371
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
Update models in the README #946
Conversation
|
Should we also add Time-LLM? HINT? |
Thanks for this! We should wait a few days; we are adding the iTransformer and MLPMultivariate. Can you also add them once they are in the main branch? |
I think we can move forward with this now and add BiTCN (#958). |
Added more models:
Not sure about what to do with HINT, since it basically wraps other models. TiDE can also be added when merged. |
We just merged DeepNPTS (#990) |
@jmoralez Added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @candalfigomoro! Small comments:
- Probabilistic forecasts are not supported for multivariate models.
- Missing TSMixerx
@elephaint I think |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Agree, that's a bit an odd one. I'm playing with more of these types of weird ones, so it's still on my list. I think maybe a 'meta-model' category would make sense sometime. |
Add missing models to the README file: