-
Notifications
You must be signed in to change notification settings - Fork 36
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
Trim unsupported fields from the body while making groq queries #47
Comments
Thanks for filing @swiftugandan. Would you be open to contributing a PR? Happy to walk you through where to make changes in the code. |
Yes sure, where should I look? |
We likely need to add logic in Currently BTW, are you passing these params in and hoping that they're stripped, or are they being inserted automatically somewhere? If the latter, maybe to start, we remove these parameters if they're specified, but |
I think they are being added as default params automatically, let me run the proxy to verify what is being sent |
I have successfully run the node proxy, but when I send a groq message, I am getting an error braintrust-proxy/apis/node/src/login.ts Line 19 in b87c35a
I reproduced the error using curl.
How can I get past this step? |
The node proxy does not currently support "api token passthrough", which the cloudlfare one does. Could you debug locally using the cloudflare proxy? Just cd into
and it should work |
it turns out no additional fields are being injected. The problem is with the messages array that is being sent. groq expects |
Very interesting. We could do something similar to what I suggested above, which per- |
Groq does not support some of the fields that are set in the body see: https://console.groq.com/docs/openai
The openai js library automatically adds some of these fields, resulting in 400 errors.
Please help trim these out for groq requests
The text was updated successfully, but these errors were encountered: