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.

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

[textanalytics] feature/ta-language-beta #24195

Merged
merged 4 commits into from
Apr 26, 2022
Merged

Conversation

kristapratico
Copy link
Member

@kristapratico kristapratico commented Apr 26, 2022

Merging the feature branch to main. No new changes, all code already reviewed.

* regenerate ta 2022-03-01-preview

* add back v3.2-preview.2 to uncomplicate things for now
* language api compat code changes

* use language api for sync/async detect_language

* use language api for sync/async recognize_entities

* use language api for sync/async recognize_pii_entities

* use language api for sync/async recognize_linked_entities

* use language api for sync/async extract_key_phrases

* use language api for sync/async analyze_sentiment

* add recordings
* initial work, not all tests passing

* fixes for tests and recordings

* fix import

* fix pylint

* skip missed tests affected by modelVersion service bug

* add healthcare entities mapping to language + tests/recordings

* lint
@azure-sdk
Copy link
Collaborator

API change check for azure-ai-textanalytics

API changes have been detected in azure-ai-textanalytics. You can review API changes here

API changes

+     property display_name: str    # Read-only
+     V2022_03_01_PREVIEW = "2022-03-01-preview"
+             display_name: Optional[str] = ..., 
+     property display_name: str    # Read-only
+             display_name: Optional[str] = ..., 

@kristapratico kristapratico merged commit ffdee53 into main Apr 26, 2022
@kristapratico kristapratico deleted the feature/ta-language-beta branch April 26, 2022 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants