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

[Anomaly Detector] Regenerate and remove manual parts #14912

Merged
merged 4 commits into from
Apr 16, 2021
Merged

[Anomaly Detector] Regenerate and remove manual parts #14912

merged 4 commits into from
Apr 16, 2021

Conversation

joheredi
Copy link
Member

Regenerating using the latest Generator version and removing manual parts as they are now auto generated. Keeping the constructor to be able to support Key credential as it is not autogenerated yet.

listMultivariateModel(options?: ListMultivariateModelOptionalParams): Promise<AnomalyDetectorClientListMultivariateModelResponse>;
listMultivariateModelNext(nextLink: string, options?: ListMultivariateModelNextOptionalParams): Promise<AnomalyDetectorClientListMultivariateModelNextResponse>;
trainMultivariateModel(modelRequest: AnomalyDetectorClientModelInfo, options?: TrainMultivariateModelOptions): Promise<AnomalyDetectorClientTrainMultivariateModelResponse>;
export class AnomalyDetectorClient extends AnomalyDetectorClientBase {
Copy link
Member Author

Choose a reason for hiding this comment

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

@lmazuel - FYI this is the API surface. The remaining manual code is just a class extending the generated client to add support for Key Credential.

The operations are all in the generated client AnomalyDetectorClientBase, no change in the name of the operations or return types.

Copy link
Member

@deyaaeldeen deyaaeldeen left a comment

Choose a reason for hiding this comment

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

As far as I can tell, this looks good! It is exciting we can rely on code gen for this much for a Track 2 package! I left a couple comments.

@joheredi joheredi merged commit 5f540a5 into Azure:master Apr 16, 2021
jay-most pushed a commit to jay-most/azure-sdk-for-js that referenced this pull request Apr 26, 2021
* regenerate and remove manual parts

* Update samples

* Regenerate names

* update command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants