D Bersan

Had same issue. > comment the line “add_compile_options(-std=c++11)” in CMakeLists.txt, > [this commit](https://github.com/felixendres/rgbdslam_v2/pull/97) breaks rgbdslam, the process segment faults at start because pcl has to be compiled with c++11 Ok,...

I was able to compile and run this following [this tutorial](https://github.com/felixendres/rgbdslam_v2/wiki/Instructions-for-Compiling-Rgbdslam-(V2)-on-a-Fresh-Ubuntu-16.04-Install-(Ros-Kinetic)-in-Virtualbox), written by the author. I believe using ROS kinetic + Ubuntu 16.04 will invariably yield this error, since it...

I also made some modifications to support SavedModel in my fork... I was going to PR and then saw this one :laughing: It seems you implemented the loading function on...

By the way, answering @serizba > What are the differences and the advantages of SavedModel over the original model of this repository? There are a lot of new possibilities, and...

> @dhiegomaga The only reason why I put the implementation into header was it is short. Apparently, it would be better to separate the implementation into a cpp file if...

Ok, I'll just try to make my point more clear: _I think there shouldn't be a new `SavedModel` class until Tensorflow 2 is released for C API._ Models saved using...

It's not an API design flaw, `SavedModel` is just another way to save a model, like *.pb files. All the methods are the same, you are even able to inherit...

Sure, that is correct. But Tensorflow 2 will be released soon, and this class would need to be completely rewritten, and not inherit from `Model`. I would prefer to have...

I don't think a macro is necessary because v2 is backwards compatible, AFAIK. I use the v2 nightly build for C API myself and it works fine with the current...

> Hi! > Thanks for your explanations. > > Actually, right now I have started to work on a sort of cppflow2 to introduce some of the changes of Tensorflow...