Skip to content

[Text Analytics] Add a sample for model versions#19087

Merged
deyaaeldeen merged 3 commits intoAzure:mainfrom
deyaaeldeen:textanalytics/model-version-sample
Dec 9, 2021
Merged

[Text Analytics] Add a sample for model versions#19087
deyaaeldeen merged 3 commits intoAzure:mainfrom
deyaaeldeen:textanalytics/model-version-sample

Conversation

@deyaaeldeen
Copy link
Copy Markdown
Member

@assafi asked for a sample to set model versions so this PR adds one.

/**
* Specify the model version by setting this property. "latest" indicates
* the latest generaly availability version of the model. When not specified,
* latest will be assumed. Model versions are date based, e.g "2021-06-01".
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe include the service doc that shows all the available modelVersions for each endpoint?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Great point, I added a link to the list of versions for NER. I could not find the versions list for all features in one place. @aahill do you know where can I find such list?

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We are just about to publish one. It's a chicken-egg situation ;)

Copy link
Copy Markdown
Contributor

@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.

I missed this request, was this for all languages or just js?

@deyaaeldeen
Copy link
Copy Markdown
Member Author

@kristapratico it is in a private email thread with the docs team and Assaf and they want to explore adding a sample for this. I will let you know if they want to proceed with it.

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.

4 participants