Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Text analytics warnings #11827

Merged
merged 2 commits into from
May 6, 2020
Merged

Text analytics warnings #11827

merged 2 commits into from
May 6, 2020

Conversation

maririos
Copy link
Member

@maririos maririos commented May 5, 2020

Service version 3.0 includes warnings at the document level. See swagger.

This PR implements the design of the warnings based on the proposal made by @annelo-msft in #11674

Not included:

@iscai-msft iscai-msft self-requested a review May 6, 2020 17:03
@maririos @annelo-msft
Update sdk/textanalytics/Azure.AI.TextAnalytics/tests/samples/Sample3…
ce04075 
…_ExtractKeyPhrasesWithWarnings.cs

Co-authored-by: Anne Thompson <annelo@microsoft.com>
@maririos maririos merged commit fbea2dd into Azure:master May 6, 2020
@maririos maririos mentioned this pull request May 6, 2020
Closed
@maririos maririos deleted the warnings branch August 27, 2020 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@annelo-msft annelo-msft annelo-msft approved these changes

@iscai-msft iscai-msft iscai-msft approved these changes

@willmtemple willmtemple Awaiting requested review from willmtemple

@mssfang mssfang Awaiting requested review from mssfang

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@maririos @annelo-msft @iscai-msft