integrating byteweight into bap. about bap HOT 3 CLOSED

I agree with Ivan. In addition, we should think about how, architecture-wise, we want to split the training and classification part. Anyone using byteweight will probably want to do their own training, test accuracy (against symbols), and so on. Once they have a trie, they will want to use it. So one question is in the library, which tree do we load? Do we make someone specify, or is there a default? These are questions you two should resolve as well.

from bap.

train is a program that we already have. It can be called to obtain signatures (it is not yet added to oasis, so that it wouldn't build automatically, see #23, but we can assume that it is already added).
And currently, byteweight comes as an executable, it doesn't provide a library level interface. We decided to move in a small steps: first to make it work as it is, and then split into parts, refactoring something useful. For example, I'm tempting to grab trie implemetation to Bap_types.

from bap.

I agree that we should split bw to application and libraries. In terms of
customized signature file, I propose that we support it in application but
not in library. This is because as one of the libraries in BAP, we only use
the signature that BAP generates. In this case one can consider BAP as a
user of bw. Similarly for training, I think we should regard it as an
application as well.
On Nov 26, 2014 10:27 AM, "Ivan Gotovchits" [email protected]
wrote:

train is a program that we already have. It can be called to obtain
signatures (it is not yet added to oasis, so that it wouldn't build
automatically, see #23
#23, but we can
assume that it is already added).
And currently, byteweight comes as an executable, it doesn't provide a
library level interface. We decided to move in a small steps: first to make
it work as it is, and then split into parts, refactoring something useful.
For example, I'm tempting to grab trie implemetation to Bap_types.

—
Reply to this email directly or view it on GitHub
#22 (comment)
.

from bap.