-
Notifications
You must be signed in to change notification settings - Fork 111
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 README and config files with Moirai-1.1-R and fix Moirai-1.0-R model weights #111
Conversation
Hi @gorold, Sorry, I don't quite understand. How these changes with torch.roll can help to fix the weights? We still have / dim in our codebase? uni2ts/src/uni2ts/module/ts_embed.py Line 178 in 27616d9
|
|
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.
Now I got it. Do you think we should test all models and see if we can get the similar results on all datasets?
Ran it on the PF benchmark ex weather: Smallwith fix:
without fix:
Feel free to test on other sizes and datasets. You can directly download the model in the PR branch with the
Note that the PR number might be different for the other model sizes. |
moirai-1.0-R-Basewith fix:
without fix:
|
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
Made PRs on huggingface to fix the regression of Moirai-1.0-R model weights with the recent code changes
https://huggingface.co/Salesforce/moirai-1.0-R-small/discussions/7
https://huggingface.co/Salesforce/moirai-1.0-R-base/discussions/6
https://huggingface.co/Salesforce/moirai-1.0-R-large/discussions/6
Please help to test, I only managed to run some small tests on my local :)
The changes to model weights were just: