Fantom-foundation / lachesis-rs

Lachesis BFT consensus for permission-less networks, in Rust
MIT License
34 stars 5 forks source link

Create benchmark testing #25

Open AgustinCB opened 5 years ago

AgustinCB commented 5 years ago

See cargo bench in rust nightly

Maxime2 commented 5 years ago

I am working on this issue. Current work in progress is here: https://github.com/Fantom-foundation/lachesis-rs/tree/bench

Maxime2 commented 5 years ago

Status update: currently it fails on inserting test hashgraph with an error message about unsigned event. Basically we need a public function which adds a transaction on the node. At the moment, create_new_head() function in swirlds.rs creates new event with empty transaction vector - we need to add a poll for transactions waiting to be added from which transactions would be drawn by create_new_new() function when it creates a new head. And thus required function would add a transaction into that transaction pool on the node. @AgustinCB what are your thoughts on it?

AgustinCB commented 5 years ago

I have that work in a branch with all the work related with real work apps. Will be merged soon.

I fail to see, though, how that can cause the error of unsigned event. Can you post some code to reproduce, please?

Thanks!