Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upReshape offset_re_shape has incorrect value #261
Comments
|
Hi, Lutz, Thanks for supporting Tengine!
04b6791 has changed the TM2_ReshapeParam here: The model on google drive used the previous format. So it makes sense with You could use this model: https://github.com/pierricklee/tmfile-sample/blob/master/detect_tflite.tmfile @LEMON-LM Please update this model to google drive. |
|
@pierricklee Users will have copies of the old file and the same happens for other files. Users might use older versions of Tengine and might still open older files in Netron which is now going to crash when the old format is loaded. You can't call it the old format because it looks identical to the new format and tools can't tell the difference. This is not acceptable. If you change the format and want things to continue working you need to update version numbers (either the version for the operator or the version number for the entire model) so tools can support both versions. Tools can't be crashing because there is no way to handle this. |
|
@lutzroeder That is a good suggestion! We are planning add an opset version and magic number into tmfile model. This will be on Tengine roadmap as soon as possible. I will keep you updated. |
|
@pierricklee |
|
@lutzroeder That's the point. We are rewriting the |


Model zoo file
detect_tflite.tmfilecontains aReshapeoperator at address23244.offset_t_param=23220points to thisTM2_ReshapeParamdata:0x0000005Bis not a valid pointer foroffset_re_shape?04b6791 seems to have changed the serialization format without updating the version number?
@pierricklee @LEMON-LM @@BUG1989