-
Notifications
You must be signed in to change notification settings - Fork 84
eng, add openai-typespec lib #3268
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?
Conversation
typespec-extension/package.json
Outdated
| }, | ||
| "dependencies": { | ||
| "@autorest/codemodel": "~4.20.1", | ||
| "@azure-tools/openai-typespec": "^1.6.1", |
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.
In short-term, we still add it to emitter dependencies, to be synced to SDK repo.
| "azure-ai-projects", # wait for service dev update tsp-location.yaml | ||
| "azure-ai-agents-persistent", # tspconfig does not have java |
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.
They likely would still fail, if run.
But in SDK repo they renamed tsp-location.yaml to _tsp-location.yaml, so it won't run.
| - job: Generate_SDK | ||
|
|
||
| timeoutInMinutes: 60 | ||
| timeoutInMinutes: 120 |
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.
The running time seems not very stable. I see range from 40min to more than 1hour (where the job times out).
Let's see if increase here would work.
foundry is using this lib for API/models from native openai https://github.com/Azure/azure-rest-api-specs/blob/feature/foundry/package.json#L4