-
Notifications
You must be signed in to change notification settings - Fork 310
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
Minimal support for Mistral 7B model. #1528
Conversation
Hello, Yeah that what I did, but I create more operation for sliding instead of using split. I will push this with chunking of input. |
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.
I think we need to assign self.sliding_window of spec to be added in model later.
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.
Where is it used later? but yes can't hurt to add it.
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.
I think to add a variable into model.bin, the variable sliding_window have to be a member of class spec, if it is not, it will not be added to model in convert process.
closing this one, reviewing #1524 with @minhthuc2502 taking into account the input chunking. |
This reverts commit 50e9ba4.
This reverts commit 8090f5f.
@minhthuc2502 Maybe check this code