Closed varunagrawal closed 2 years ago
Also as an FYI, JsonSaver.h
has a license referring to Holomatic. We should probably update it or remove it to avoid potential legal issues?
Uhm, #337 ? Also, you should really carefully think about including things like boost::format in headers.
Huh didn't see that, thank you! I figured that JsonSaver.h already had a boost import so adding another wouldn't be too bad?
Headers should be as specific as possible, esp. boost headers, because they can substantially increase compile times.
Recent updates from GTSAM now require us to explicitly include
boost/format.hpp
.