Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

there is something wrong about the main() function in mxnet/tools/caffe_converter/convert_symbol.py #4121

Closed
animebing opened this issue Dec 6, 2016 · 4 comments

Comments

@animebing
Copy link

In the main() function, the return of proto2script(sys.argv[1]) should be 3 values, not 2 values

@oura-fj
Copy link
Contributor

oura-fj commented Dec 7, 2016

proto2script(sys.argv[1]) returns 3 values, so I think you should chage the code
symbol_string, output_name = proto2script(sys.argv[1])
to
symbol_string, output_name, input_dim = proto2script(sys.argv[1])

@animebing
Copy link
Author

@oura-fj Thanks, this is what I have done. Maybe the code in github should be modified to avoid the mistake

@oura-fj
Copy link
Contributor

oura-fj commented Dec 7, 2016

@animebing Yes, I think so too. I modified the code in github and sent a pull request.

@piiswrong
Copy link
Contributor

should have been fixed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants