feat: Add TextAnalyze transformer to add support for Text Analytics /analyze endpoint
#1267
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
stuartleeks
changed the title
/analyze endpoint/analyze endpoint
stuartleeks
force-pushed
the
sl/text-analytics-analyze
branch
2 times, most recently
from
26dcc7e to
c0da2a4
Compare
stuartleeks
changed the title
/analyze endpoint/analyze endpoint
stuartleeks
force-pushed
the
sl/text-analytics-analyze
branch
from
c0da2a4 to
5cb9f77
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Codecov Report
@@ Coverage Diff @@
## master #1267 +/- ##
==========================================
+ Coverage 83.28% 83.30% +0.02%
==========================================
Files 300 300
Lines 13720 13828 +108
Branches 670 672 +2
==========================================
+ Hits 11427 11520 +93
- Misses 2293 2308 +15
Continue to review full report at Codecov.
|
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
stuartleeks
force-pushed
the
sl/text-analytics-analyze
branch
3 times, most recently
from
f1b1192 to
44fb0db
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
stuartleeks
force-pushed
the
sl/text-analytics-analyze
branch
from
44fb0db to
8b00517
Compare
|
/azp run |
|
Commenter does not have sufficient privileges for PR 1267 in repo microsoft/SynapseML |
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
stuartleeks
force-pushed
the
sl/text-analytics-analyze
branch
from
8b00517 to
73370f4
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
mhamilton723
reviewed
Dec 2, 2021
| sentimentAnalysis: Option[Seq[TAAnalyzeResultTaskResults[SentimentScoredDocumentV3]]]) | ||
|
|
||
| object TAAnalyzeResults extends SparkBindings[TAAnalyzeResult] | ||
|
|
There was a problem hiding this comment.
nit: This file has a bit of extra whitespace in a few areas
mhamilton723
force-pushed
the
sl/text-analytics-analyze
branch
from
679b7dd to
2e9c4a9
Compare
|
/azp run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
mhamilton723
approved these changes
Dec 2, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The Text Analytics
/analyzeendpoint provides a way to perform multiple tasks in a single API call (good for high-throughput scenarios)