-
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
[TextAnalytics] Added sample for Extractive Text Summarization #23097
[TextAnalytics] Added sample for Extractive Text Summarization #23097
Conversation
Get a summary for the input documents by extracting their most relevant sentences. Note that this API can only be used as part of an [Analyze Operation](#run-multiple-actions-asynchronously). | ||
|
||
```C# Snippet:TextAnalyticsExtractSummaryConvenienceAsyncAll | ||
// Get input document. |
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.
Adding monster-size input I stole from JS. Running this API for long documents seems more appropriate.
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.
Def a nit: I think we can achieve the same goal with a smaller doc. No? maybe remove some paragraphs? at least for the mean Readme. it just looks weird.
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.
Created #23105 for tracking.
sdk/textanalytics/Azure.AI.TextAnalytics/samples/Sample8_ExtractSummary.md
Outdated
Show resolved
Hide resolved
sdk/textanalytics/Azure.AI.TextAnalytics/samples/Sample8_ExtractSummary.md
Show resolved
Hide resolved
Get a summary for the input documents by extracting their most relevant sentences. Note that this API can only be used as part of an [Analyze Operation](#run-multiple-actions-asynchronously). | ||
|
||
```C# Snippet:TextAnalyticsExtractSummaryConvenienceAsyncAll | ||
// Get input document. |
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.
Def a nit: I think we can achieve the same goal with a smaller doc. No? maybe remove some paragraphs? at least for the mean Readme. it just looks weird.
sdk/textanalytics/Azure.AI.TextAnalytics/samples/Sample8_ExtractSummary.md
Outdated
Show resolved
Hide resolved
sdk/textanalytics/Azure.AI.TextAnalytics/samples/Sample8_ExtractSummary.md
Outdated
Show resolved
Hide resolved
…23109) * [TextAnalytics] Generating code from 3.2-preview.1 swagger (#22774) * [TextAnalytics] Implemented ExtractiveSummarization input (#22791) Co-authored-by: Mariana Rios Flores <[email protected]> * [TextAnalytics] Added DisableServiceLogs property to ExtractSummaryAction (#22803) * [TextAnalytics] Remaining Extractive Text Summarization implementation work (#22841) Co-authored-by: Deyaaeldeen Almahallawi <[email protected]> * [TextAnalytics] Added model factory for extractive summarization models (#22873) * [TextAnalytics] SummarySentece is now a struct (#22915) * [TextAnalytics] Recording old tests with service version 3.2-preview.1 (#23078) * [TextAnalytics] Added sample for Extractive Text Summarization (#23097) Co-authored-by: Mariana Rios Flores <[email protected]> Co-authored-by: Deyaaeldeen Almahallawi <[email protected]>
Fixes #22941.