Skip to content

Add new SDK for Microsoft.MachineLearning/webservices RP#1987

Merged
hovsepm merged 23 commits intoAzure:AutoRestfrom
tudordid:AutoRest
May 10, 2016
Merged

Add new SDK for Microsoft.MachineLearning/webservices RP#1987
hovsepm merged 23 commits intoAzure:AutoRestfrom
tudordid:AutoRest

Conversation

@tudordid
Copy link
Copy Markdown
Contributor

First iteration of the .NET sdk code for the MachineLearning/ webservices SDK

@azuresdkci
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@azurecla
Copy link
Copy Markdown

Hi @tudordid, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!


It looks like you're working at Microsoft (tudordid). If you're full-time, we DON'T require a contribution license agreement.



If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@hovsepm
Copy link
Copy Markdown
Contributor

hovsepm commented May 10, 2016

@azuresdkci add to whitelist

Copy link
Copy Markdown
Contributor

@hovsepm hovsepm May 10, 2016

Choose a reason for hiding this comment

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

should not be from your fork. You should generate the sources from Azure github repo #Closed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Correct, I forgot to update this after we have pushed the swagger spec in the main repo. Will fix.

@hovsepm
Copy link
Copy Markdown
Contributor

hovsepm commented May 10, 2016

@tudordid you will need to pull from Azure fork AutoRest branch and resolve conflicts

tudordid added 20 commits May 10, 2016 15:26
  - addition of type property to GraphParameter
  - removal of not needed API /subscriptions/{subscriptionId}/providers/Microsoft/MachineLearning/checkNameAvailability
 - fixes to documentation of classes
- added utility to support serialization / deserialization of models
- refactored model tests
- added *Additions classes to generated code to inject correct request id in error responses for async operations (temp fix)
- renamed projects to be ML generic, as we plan to host the workspaces SDK code here as well, not just web services
- changed test code to also create a commitment plan as part of test execution
  - renamed WebServicePropertiesForGraphPackage class to GraphPackage
  - made several properties required
- Re-recorded tests against Dogfood with dynamic creation of CP resource now + more cleanup
tudordid added 3 commits May 10, 2016 15:28
 - fixed generation script to use swager spec from Azure repo
 - changed AssemblyInfo.cs file to show correct assembly version
 - removed blank lines
@hovsepm hovsepm merged commit 68df050 into Azure:AutoRest May 10, 2016
kinelski added a commit that referenced this pull request Nov 14, 2023
…in (#39965)

* Initial generated project (#1987)

* Added admin live tests for classifiers (#2012)

* Added admin live tests for models (#2015)

* Misc operations + classify live tests (#2017)

* Added admin live tests for misc operations

* Added main live tests for classifiers

* Generated from latest spec + added recordings (#39936)

* Generated from the latest spec

* Added recordings
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.

4 participants