Feature request: clone method about rbush HOT 6 OPEN

Yes, that's exactly what I'd want!

from rbush.

To keep the references, I believe current way to do it is:

function clone(tree) {
  const cloned = Rbush()
  cloned.load(tree.all())
  return cloned
}

from rbush.

Thanks for the request! Would you expect RBush to clone the tree structure but don't clone the items from the original input array (and keep references instead)?

from rbush.

Great! If you want to tackle this, I'm happy to review a PR.

from rbush.

@Pyrolistical this would be sub-optimal due to the need to perform loading again — in theory, this step could be avoided with selective tree cloning. But this probably won't be a bottleneck anyway.

from rbush.

Understood. I was just showing how you would do it. Its not great.

Maybe if you refactored to use a persistent data structure then you can have zero cost clone.

from rbush.