search

serizba / cppflow

Run TensorFlow models in C++ without installation and without Bazel
https://serizba.github.io/cppflow/
MIT License
779 stars 177 forks source link

yet another frozen graph support #152

Closed bytosaur closed 2 years ago

bytosaur commented 2 years ago

Hey there,

I took some ideas from #145 and this issue from @natxopedreira to do some changes to the model class in order to support frozen graphs. I haven't tested everything but it seemed that these changes don't break other stuff.

@serizba what do u think in general about supporting frozen graphs?

bytosaur commented 2 years ago

hey @serizba,

just bumbing it up just to check whether it makes sense to support this feature in cppflow2. I do understand it s legacy, but can be useful to some ppl :)

serizba commented 2 years ago

Hi @bytosaur

Sorry. I forgot about this. Yeah, it may be a good idea and it may be helpful. I will give it a look!

Thanks a lot for your effort!

serizba commented 2 years ago

This is a good, work, it adds legacy functionality without changing anything for the rest of the users. I think it will be helpful. I will merge it.

natxopedreira commented 2 years ago

Genial!!!

I found really useful to have old model compatibility support.