Skip to content

Conversation

@codemzs
Copy link
Member

@codemzs codemzs commented Aug 2, 2019

  • Rewired Microsoft.ML.Tensorflow to use Tensorflow .NET C# bindings.
  • Introduces Microsoft.ML.Dnn package that contains transfer learning based image classification API using resnet and inception V3 models.
    • Contains API to retrain a DNN model using tensorflow.

All tensorflow tests are passing. The current test failures have to do with ONNX tests that seem to be using a higher version of protobuf since TF.NET also uses protobuf.

Copy link
Contributor

@CESARDELATORRE CESARDELATORRE left a comment

Choose a reason for hiding this comment

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

I'm really excited to try this with more elaborated samples as soon as we have the NuGet package. I provided some feedback with suggestions and possible improvements, but I consider this code could already be merged and keep evolving it from there taking my comments into account, since this still will be a very early preview.

The sooner we can create samples and code like what users would be creating, the better for being able to provide feedback and evolve accordingly. 👍

@Oceania2018
Copy link

Oceania2018 commented Aug 5, 2019

@CESARDELATORRE We will add more other Model Architectures like MobileNet, ResNet and NASNet, which could give you some extra precision. I will implement it in TF.NET first. Hope it be true for ML.NET v1.4 release. #Resolved


if (!File.Exists(options.ModelLocation))
{
if (options.Arch == Architecture.InceptionV3)
Copy link
Contributor

@CESARDELATORRE CESARDELATORRE Aug 5, 2019

Choose a reason for hiding this comment

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

This is great, I see you added InceptionV3 over the weekend! 👍 #Resolved

@codemzs
Copy link
Member Author

codemzs commented Aug 6, 2019

@Oceania2018 This release we are not advertising GPU support. I want to first test it before we remove this dependency.

@codemzs
Copy link
Member Author

codemzs commented Aug 6, 2019

Thanks, @CESARDELATORRE for reviewing.

@codemzs codemzs merged commit 8d51eee into dotnet:master Aug 6, 2019
@ghost ghost locked as resolved and limited conversation to collaborators Mar 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants