<div class="gmail_quote">On Sat, Oct 15, 2011 at 10:12, Vijay S. Mahadevan <span dir="ltr"><<a href="mailto:vijay.m@gmail.com">vijay.m@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div id=":3ed">Find attached the bundle with the changes for<br>
VecNestSetSubVec/VecNestSetSubVecs. The test file and the outputs are<br>
also included in the bundle. Feel free to modify the documentation if<br>
need be.<br></div></blockquote><div><br></div><div>Thanks, applied.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div id=":3ed">
<br>
The only thing that I think needs a better implementation is the<br>
SetSubVecs where I've just looped over the indices and recursively<br>
called the single SetSubVec. This was hard to optimize since<br>
accommodating variable vectors involves traversing the IS one by one<br>
and I couldn't figure out a way to implement this concurrently.<br>
Anyway, if you can think of something better, let me know.</div></blockquote></div><br><div>Let's wait on optimizing until we have a use case where it matters. Since everything else expects data structures to be rebuilt if anything changes, I don't think it's a concern just yet. If we allow more resizing in the future, we can optimize this routine.</div>