search

C5T / Current

C++ framework for realtime machine learning.
https://medium.com/dima-korolev/current-for-realtime-machine-learning-4f04aa8ab81a
97 stars 29 forks source link

Using the "new" `current_build_info.h`. #970

Closed dimacurrentai closed 7 months ago

dimacurrentai commented 7 months ago

Fingers crossed this passes the GitHub PR guard now.

If yes, I'll remove the "old" current_build.h target and merge.

dimacurrentai commented 7 months ago

Okay, it did not "work" as intended right away, BUT: I've managed to re-build this target using cmake means only, 100% of the time.

AND: I have double-checked that only the targets that directly #include "current_build_info.h" do get re-build every time.

Which is the ideal outcome as far as I can tell: whatever #include-s current_build_info.h gets rebuilt every time with the freshly re-generated current_build_info.h, whereas whatever does not #include current_build_info.h is kept up to date.

Of course, I intend to only require linking whatever needs current_build_info.h, with the .cc units that #include current_build_info.h to be as compact as possible. Amen.

dimacurrentai commented 7 months ago

Yay, all green, merging!