-
Notifications
You must be signed in to change notification settings - Fork 2.1k
doc update to PR #2528 and issue #1508 #3150
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: dev
Are you sure you want to change the base?
doc update to PR #2528 and issue #1508 #3150
Conversation
provides an overview of the optional options to allow opencode to use different azure ai url styles
- How this options will change the generated Azure url to be used in opencode | ||
|
||
| useDeploymentBasedUrls | useCompletionUrls | Generated URL | | ||
|------------------------|-------------------|-------------------------------------------------------------------------------| | ||
| false | false | %baseURL%/v1/responses?api-version=%apiVersion% | | ||
| true | false | %baseURL%/deployments/%Model%/responses?api-version=%apiVersion% | | ||
| false | true | %baseURL%/v1/responses?api-version=%apiVersion% | | ||
| true | true | %baseURL%/deployments/%Model%/chat/completions?api-version=%apiVersion% | |
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 jay may have some opinions on the docs, but do you think we could tweak this a tad?
Correct me if you disagree:
I was thinking instead of just showing resolved url I wonder if it'd be good to add a note about what kinda deployment requires the need for this config setting?
So like:
if you have X deployment, you will need to set Y so opencode can hit correct endpoint
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.
@rekram1-node i did some testing, it seems the provided url is somewhat tied to the selected model. for example:
gpt-5-chat => deployment | completion => https://xxxx/deployments/gpt-5-chat/chat/completions?api-version=xxx
gpt-5-mini => none? | responses => https://xxxx/responses?api-version=xxx
DeepSeek-R1 => none? | completion => https://xxx/models/chat/completions?api-version=xxx
since you can not influence the url azure provides you i think it is best to have it in the format i proposed, or at least i don't see another good way to give the most useful information. with this table you can at least check the azure provided url and see what setting might work for your case
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.
not sure if it would make more sense to for azure to actually set the url in the model configuration by this point - at last from what i see and tested they have to many url formats they use - not sure if this is even possible with the current configuration usage .. just thinking out loud at the moment.
if i have time later, i'll do some more testing with different models in Azure
provides an overview of the optional options to allow opencode to use different azure ai url styles
PR #2528
Issue #1508
@rekram1-node have a look if this is acceptable as an addition to the docs for the above described PR / Issue