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

Tensorflow loader #2423

Open
dati91 opened this issue Feb 21, 2019 · 3 comments

Comments

Projects
None yet
4 participants
@dati91
Copy link

commented Feb 21, 2019

Hi!

I started to look into this project and seems really cool. But I noticed it only supports Caffe2 and ONNX formats.
Will it support other loaders e.g. Tensorflow? Is it on the roadmap?

Thanks in advance!

@jackm321

This comment has been minimized.

Copy link
Contributor

commented Feb 21, 2019

I'm not aware right now of any plans to support Tensorflow importing directly but you may be able to convert a Tensorflow graph to ONNX using https://github.com/onnx/tensorflow-onnx and then import the ONNX model.

@dati91

This comment has been minimized.

Copy link
Author

commented Feb 21, 2019

@jackm321 Thanks for the quick reply!

I actually looked into that project and another ( https://github.com/Microsoft/MMdnn ), but I couldn't manage to convert it correctly to load into Glow.

But I looked into the Caffe2Loader code and seemed pretty straight forward, so if I decide to use Glow for my project maybe I can contribute that to this awesome project.

@tlepley-cadence

This comment has been minimized.

Copy link
Contributor

commented Feb 21, 2019

@dati91 We have been using tf2onnx (the project pointed out by @jackm321) and it works well, even if we need to fix issues with the converter and/or the Glow ONNX loader some time to time. But having a native TensorFlow loader in Glow would be in my opinion a nice to have. Is it what you have in mind when you speak about contributing to Glow ?

@nickgg nickgg added the wishlist label Apr 29, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.