You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Every hash it needs is already right there! here is the call to add_path that forms the core of of_ledger_subset_exn, and here is the definition of add_path. Note the calls to Hash.merge - these are expensive and wasteful. Instead, we could augment the notion of path to include the merge hashes.
I've started some work on the fix/no-sparse-ledger-hashing branch that does this. merkle_path_full needs to be implemented for all the relevant ledgers.
The text was updated successfully, but these errors were encountered:
Every hash it needs is already right there! here is the call to
add_path
that forms the core ofof_ledger_subset_exn
, and here is the definition ofadd_path
. Note the calls toHash.merge
- these are expensive and wasteful. Instead, we could augment the notion of path to include the merge hashes.I've started some work on the fix/no-sparse-ledger-hashing branch that does this.
merkle_path_full
needs to be implemented for all the relevant ledgers.The text was updated successfully, but these errors were encountered: