-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Prepare Conversations Language Understanding SDK 1.1.0-beta.1 #29144
Conversation
Caused by Azure#29140 (seemingly)
Works around Azure#29141 and Azure#29143
/cc @annelo-msft in case you want to check out any of the DPG stuff. |
API change check APIView has identified API level changes in this PR and created following API reviews. |
sdk/cognitivelanguage/Azure.AI.Language.Conversations/README.md
Outdated
Show resolved
Hide resolved
.../Azure.AI.Language.Conversations/samples/Sample6_AnalyzeConversation_ConversationPII_Text.md
Outdated
Show resolved
Hide resolved
...ognitivelanguage/Azure.AI.Language.Conversations/src/Generated/ConversationAnalysisClient.cs
Show resolved
Hide resolved
sdk/cognitivelanguage/Azure.AI.Language.Conversations/src/Models/AnalyzeError.cs
Outdated
Show resolved
Hide resolved
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'd love to see tests that call the protocol methods on the DPG client. Thanks!
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.
Looks good, mostly nits
sdk/cognitivelanguage/Azure.AI.Language.Conversations/README.md
Outdated
Show resolved
Hide resolved
sdk/cognitivelanguage/Azure.AI.Language.Conversations/src/ConversationAnalysisProjectsClient.cs
Outdated
Show resolved
Hide resolved
sdk/cognitivelanguage/Azure.AI.Language.Conversations/src/ConversationAnalysisProjectsClient.cs
Outdated
Show resolved
Hide resolved
...nguage/Azure.AI.Language.Conversations/api/Azure.AI.Language.Conversations.netstandard2.0.cs
Show resolved
Hide resolved
...nguage/Azure.AI.Language.Conversations/api/Azure.AI.Language.Conversations.netstandard2.0.cs
Show resolved
Hide resolved
...nguage/Azure.AI.Language.Conversations/api/Azure.AI.Language.Conversations.netstandard2.0.cs
Show resolved
Hide resolved
…29144) * Convert to DPG with HLC models * Ignore long sync LRO test Caused by Azure#29140 (seemingly) * Add swagger transforms Works around Azure#29141 and Azure#29143 * Convert to DPG Also fixes Azure#26379 * Update public APIs and documentation * Resolve PR feedback * Resolve offline feedback * Update generated code
…29144) * Convert to DPG with HLC models * Ignore long sync LRO test Caused by Azure#29140 (seemingly) * Add swagger transforms Works around Azure#29141 and Azure#29143 * Convert to DPG Also fixes Azure#26379 * Update public APIs and documentation * Resolve PR feedback * Resolve offline feedback * Update generated code
Contains inference DPG client but convenience methods previously shipped for 1.0.0 betas, as well as PII detection and summarization.