lucid icon indicating copy to clipboard operation
lucid copied to clipboard

Consider eliminating explicit `.load_graphdef()` call?

Open ludwigschubert opened this issue 7 years ago • 2 comments

Hey @colah @znah ; should we consider automatically calling load_graphdef when instantiating a modelzoo class? To me this boils down to:

What can you currently do with an instantiated modelzoo.Model, that you couldn't do with just the class?

It feels like this could simplify the current API, but it may also hide the fact that a graph definition may need to be downloaded. Looking forward to your opinions!

ludwigschubert avatar Feb 06 '18 03:02 ludwigschubert

Original motivation for having a separate load_graphdef() function was to support some customization, like optional stripping of some layers, or having a few graph versions (say CPU/TPU) for some models. I don't think we really need it now. I support removing it and doing all customization in constructor.

znah avatar Feb 08 '18 13:02 znah

Thanks for letting me know! Will do a PR soon.

ludwigschubert avatar Feb 08 '18 22:02 ludwigschubert