alibaba / alibaba/x-deeplearning

blaze convert dense model failed because the xdl version upgrade to 1.2

Open
#254 8 comments 0 reactions 0 assignees View on GitHub
Dominant language
PureBasic
Stars
4.3k
Forks
1k
PR merge metrics
No merged PRs in 30d

Description

The `Nodetype` definition in `xdl(version=1.2) graph_def.proto` is
```
message NodeDef {
string name = 1;
string op = 2;
repeated string input = 3;
DeviceDef device = 4;
map attr = 5;
}
```

but in xdl(version=1.1) is
```
message NodeDef {
string name = 1;
string op = 2;
repeated string input = 3;
repeated DataType output_type = 4;
DeviceDef device = 5;
map attr = 6;
}
```

and in the same `graph_def.proto` of blaze is
```
message NodeDef {
string name = 1;
string op = 2;
repeated string input = 3;
DeviceDef device = 4;
map attr = 5;
}
```

So, there is a error in blaze conveting dense model.
```
[libprotobuf ERROR /home/work/open-code/x-deeplearning/blaze/thirdparty/protobuf/protobuf-3.6.0/src/google/protobuf/text_format.cc:307] Error parsing text-format xdl.proto.GraphDef: 4:14: Message type "xdl.proto.NodeDef" has no field named "output_type".
```

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by comparing the xdl 1.2 and 1.1 NodeDef definitions in graph_def.proto with Blaze's definition. Reproduce the dense-model conversion and inspect the protobuf text-format error about the missing output_type field; done means the conversion no longer fails because of this schema mismatch.

Written by the indexing model from the issue text.

Assessment

Domain
machine-learning
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.