Currrently the aggregate avl tree exposes a method from_vec to build the tree from an existing vector. It would be useful to replace this with std::iter::FromIterator.
Additionally the current implementation of from_vec builds by subsequent insertions, it would be more performant to do this in-place, avoiding erroneous allocations as well as getting a lower bound in time complexity
Currrently the aggregate avl tree exposes a method
from_vecto build the tree from an existing vector. It would be useful to replace this withstd::iter::FromIterator.Additionally the current implementation of
from_vecbuilds by subsequent insertions, it would be more performant to do this in-place, avoiding erroneous allocations as well as getting a lower bound in time complexity