[Octave-bug-tracker] [bug #63962] perms - Performance - Usage of native C++ algorithm helpful

[Arun Giridhar]

Follow-up Comment #4, bug #63962 (project octave):

In the course of reviewing your patch, I realized that the existing perms.m
does not do unique properly for cell inputs. The workaround is to construct a
numerical index vector and pass that to perms, then use it to index into the
cell array. That should ideally be done by perms though, so that's a bug. I'll
file a separate report.

Would your code be able to accommodate unique for cells by doing that sort of
numerical index translation? If not, an alternative is to do that index
conversion in perms.m, pass it to the new perms.cc, and then translate the
result back in perms.m. Thoughts?


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?63962>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/