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

Using named-tuple in OneToOneTransforms' constructor to make API more readable. #324

Merged
merged 12 commits into from
Jun 12, 2018
2 changes: 1 addition & 1 deletion src/Microsoft.ML.Data/Model/Onnx/OnnxContext.cs
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public sealed class OnnxContext
private readonly string _producerVersion;
private readonly long _modelVersion;

public OnnxContext(IHostEnvironment env, string name, string producerName,
public OnnxContext(IHostEnvironment env, string name, string producerName,
string producerVersion, long modelVersion, string domain)
{
Contracts.CheckValue(env, nameof(env));
Expand Down
2 changes: 1 addition & 1 deletion src/Microsoft.ML.Data/Model/Onnx/OnnxUtils.cs
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ public NodeProtoWrapper(NodeProto node)
}
}

public static ModelProto MakeModel(List<NodeProto> nodes, string producerName, string name,
public static ModelProto MakeModel(List<NodeProto> nodes, string producerName, string name,
string domain, string producerVersion, long modelVersion, List<ModelArgs> inputs,
List<ModelArgs> outputs, List<ModelArgs> intermediateValues)
{
Expand Down
Loading