-
Notifications
You must be signed in to change notification settings - Fork 18.7k
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
boost dependency #4727
Comments
See #2537 (comment). If you are interested in a pure c++11 solution Take a look at https://github.com/tiny-dnn/tiny-dnn |
@akosenkov |
The problematic aspect of the Boost dependency is that Caffe exposes Boost as part of its public API, instead of just being an internal implementation detail. This makes code using Caffe also depend on Boost. I'm currently integrating Caffe into a fairly large code base, and are facing problems related to Boost header version mismatches between Caffe and the rest of the code base. These types of problems are making Caffe adoption on Windows a hassle. Could it be possible to revisit this issue of replacing some/most of Boost with C++11 now? C++11 is widely supported on most platforms now. My impression is that the subset of C++11 used by Caffe have been supported GCC since 4.8.1 (May 2013), clang since 3.3 (June 2013), MSVC since 2013 (June 2013) & CUDA since 7 (March 2015). |
Could you please get rid of the Boost from the list of the dependencies?
There are not so many places in the code where it's actually necessary - C++11 should cover most of that.
I believe this way the library will be more attractive for the developers community and potential deployment.
The text was updated successfully, but these errors were encountered: