Skip to content

Make Onnx path parameter mandatory in OnnxConverter #423

Closed
@zeahmed

Description

@zeahmed

According to the documentation, both Onnx and Json paths are optional. Make at least Onnx path mandatory for conversion.

https://github.com/dotnet/machinelearning/blob/master/src/Microsoft.ML/Models/OnnxConverter.cs

OnnxConverter converter = new OnnxConverter()
{
    InputsToDrop = new[] { "Label" },
    OutputsToDrop = new[] { "Label", "Features" },
    Onnx = onnxPath,
    Json = onnxAsJsonPath,
    Domain = "Onnx"
};

converter.Convert(model);

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions