search

Dobiasd / frugally-deep

A lightweight header-only library for using Keras (TensorFlow) models in C++.
MIT License
1.07k stars 237 forks source link

Missing inline keyword in tensor.hpp #381

Closed Fadi-B closed 1 year ago

Fadi-B commented 1 year ago

Hi,

Thank you for the great project!

I noticed that I was getting a "redefinition" error for some functions in the tensor.hpp file when including the fdeep library in header files. The error was not present when including into source files. This issue was fixed by adding the inline keyword that seems to have been missing for some of the functions in tensor.hpp. The issue was resolved after that.

Please review the changes I have made just to make sure I have not messed anything up.

Kind Regards, Fadi

Dobiasd commented 1 year ago

Oh, what a blunder! :woozy_face:

Thanks a lot for fixing this! :heart: :+1:

Fadi-B commented 1 year ago

No worries! =)