-
Notifications
You must be signed in to change notification settings - Fork 3
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
bug: Fix, update & improve models in Jan Hub #46
Comments
Codestral 22B Q4:The response is weird: Screen.Recording.2024-10-21.at.7.25.14.PM.movScreen.Recording.2024-10-21.at.7.36.25.PM.mov |
LlaVa 7BWeird responses: Reported by user: https://zoom.us/clips/share/riUumJZ0uuzb5vQvZ2eZbMkmOq1nvU7O8VTD5FuBNtxRaO89rp9xA7CibJFCLlGju3nfyLsB_19iPegc0nSM4qxV.POPOcY7WXml_Ab8P Screen.Recording.2024-10-22.at.2.37.24.PM.movScreen.Recording.2024-10-22.at.5.23.12.PM.mov |
@imtuyethan I recommending converting the Checklist you have above, into a table so we can track the status/fixing status. Please work with @hahuyhoang411 - it may be that certain models are unsavable, and we should just remove them from the library. |
TasklistI have QA-ed all models, please check ticket description for the latest update:
|
Current hub contains a lot of outdated models, and some new models have a prompt template bug. Here is my suggestion based on @imtuyethan QA-ed list: The rationale for this delete list is model has been released >6months will be removed. Delete list:
Keep list:
|
@hahuyhoang411 Should we add more new/trending models? The list seems short for a whole model hub. Some edge cases we need to handle:We can delete them from Hub, but they still show up on the users' side if they have downloaded these legacy models. How do we inform them when these models don't work? |
I edited the list based on current trends. cc @imtuyethan |
Problem
I have encountered many issues with the wrong model default settings (incorrect prompt template, the stop words missing, etc.).
e.g., comments in Jan 0.5.7 Release Sign Off janhq/jan#3818
Model Testing Results
I have tested 45 models from Jan Hub, here are the results.
Next step
cc @hahuyhoang411
On one note
We will need to develop model.yaml to easily define model capabilities (e.g. function calling, vision, etc). Users are facing an issue with imported LlaVa: janhq/jan#3855
Related
The text was updated successfully, but these errors were encountered: