Improve summary of order process

[soxofaan]

spin-off from #409 :stuck_out_tongue:

[m-mohr]

A change here should probably be aligned with rearrange.

For me in this proposal it's not clear what the order is. This sounds a bit like whether it would return "ascending" or "descending" or "unsorted" and not the indices for the array in a sorted order ;-)

[soxofaan]

A change here should probably be aligned with rearrange.

Because rearrange uses "permutation" in its summary? I think that's fine.

For me in this proposal it's not clear what the order is. This sounds a bit like whether it would return "ascending" or "descending" or "unsorted"

Ok fair. An alternative, stealing from numpy argsort:

Get indices that would sort the array

FYI: what I don't like about the current summary "Create a permutation":

• "create a" sound like the process just randomly generates some permutation, regardless of the original data
• "permutation": it's not always obvious if "permutation" refers to the set of indices to implement the rearrangement, or if it is about the result after rearranging

so "create a permutation" roughly sounds like "randomly shuffle the array" to me

[m-mohr]

@soxofaan I made a couple of changes to the PR, especially to link between the processes. Can you check whether this works for you?

[soxofaan]

yes it's better now

apparently I can approve this PR, I guess because I started it myself

[m-mohr]

True, I've just merged it now.