-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Custom vision 3 1 #7074
Custom vision 3 1 #7074
Conversation
In Testing, Please Ignore[Logs] (Generated from c8628d2, Iteration 5)Go: test-repo-billy/azure-sdk-for-go [Logs]
Python: test-repo-billy/azure-sdk-for-python [Logs] [Diff]
Java: test-repo-billy/azure-sdk-for-java [Logs]
JavaScript: test-repo-billy/azure-sdk-for-js [Logs]
Ruby: test-repo-billy/azure-sdk-for-ruby [Logs]
|
Automation for azure-sdk-for-pythonThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-goNothing to generate for azure-sdk-for-go |
Can one of the admins verify this patch? |
"parameters": [ | ||
{ | ||
"name": "projectId", | ||
|
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.
"/projects/{projectId}/tagsAndRegions/suggestions": { |
"DomainsApi" | ||
], | ||
"summary": "Get a list of the available domains.", | ||
"operationId": "GetDomains", |
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.
normally operationId is like Resource_Action
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.
We've not been using that style of naming in cognitive serivces. Renaming the operations isn't something we'd want to do in this change as we're trying not to break existing clients.
hi @ruowan could you pls help to look at the avocado error? thanks. |
Hi, @areddish |
The PR is ready to merge after CI fixed. PR is reviewed by ARM team in private repo already. |
@yungezz - I've updated the encoding and it appears all is well with the checks. |
hi @areddish , there're still 2 errors in CI validation. could you pls do below fixing:
I'll merge it after those issues fixed and CI passed. |
Fix example
Latest improvements:
Adding v3.1 of Custom Vision Training API
Contribution checklist:
ARM API Review Checklist
Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.
Please follow the link to find more details on API review process.