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

[Language Text] Update tests to match behavior in PPE #23510

Merged
merged 4 commits into from
Oct 14, 2022
Merged

[Language Text] Update tests to match behavior in PPE #23510

merged 4 commits into from
Oct 14, 2022

Conversation

deyaaeldeen
Copy link
Member

@deyaaeldeen deyaaeldeen commented Oct 13, 2022 •
edited
Loading

And adds tests for FHIR, extractive summarization, and abstractive summarization.

@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-ai-language-text

kristapratico
kristapratico approved these changes Oct 14, 2022
View reviewed changes
Copy link
Member

@kristapratico kristapratico left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might be good to open a tracking issue to reenable AAD / tests you're currently skipping because of PPE

@deyaaeldeen deyaaeldeen merged commit c0bdd3f into Azure:feature/ai-language-text@1.1 Oct 14, 2022
@deyaaeldeen deyaaeldeen deleted the language-text/update-tests branch October 14, 2022 04:19
@deyaaeldeen deyaaeldeen restored the language-text/update-tests branch October 15, 2022 03:10
@deyaaeldeen deyaaeldeen mentioned this pull request Nov 1, 2022
Merged
deyaaeldeen added a commit that referenced this pull request Nov 2, 2022
@deyaaeldeen
[Language Text] Merge feature branch for v1.1-beta.1 work into main (#…
904e1c8 
…23673)

The work has been reviewed in the following PRs:

#23208
#23465
#23510
#23525
#23589
#23564
#23619

Live run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1959799&view=results
mpodwysocki pushed a commit that referenced this pull request Nov 2, 2022
@deyaaeldeen @mpodwysocki
[Language Text] Merge feature branch for v1.1-beta.1 work into main (#…
2dfb4cc 
…23673)

The work has been reviewed in the following PRs:

#23208
#23465
#23510
#23525
#23589
#23564
#23619

Live run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1959799&view=results
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kristapratico kristapratico kristapratico approved these changes

@mssfang mssfang mssfang approved these changes

@joseharriaga joseharriaga Awaiting requested review from joseharriaga

@minhanh-phan minhanh-phan Awaiting requested review from minhanh-phan

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

Successfully merging this pull request may close these issues.
4 participants
@deyaaeldeen @azure-sdk @kristapratico @mssfang