-
Notifications
You must be signed in to change notification settings - Fork 1.7k
qol(virtual-quota) #4672
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
base: main
Are you sure you want to change the base?
qol(virtual-quota) #4672
Conversation
…ay and image support by default
|
|
hey @chrarnoldus @mcowger need your thought and review on this PR. Thanks! |
Seems reasonable.
Nice.
Not sure I'm a fan of this. Wouldn't it result in a user experiencing confusion?
Wasn't this fixed with your last round of changes that updated the information as the model switched? |
okay it actually make sense, giving the user 404 error is not a good experience, reverted it
this is actually just a UI "improvement", at least that's what i think. the current context window is 1 by default. My case, for example: The critical one for me is actually the cooldown, my case: |
|
so any more feedback on this @mcowger @chrarnoldus ? is it possible for this PR to be merged? |
Context
Implementation
Screenshots
How to Test
Use the provider
Get in Touch
Ivanarifin