You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Probably enforces there are always kwargs, so we don't test default values to behave as expected...
Edit: this isn't true, missing kwargs are tested by hypothesis.
The problem comes from the fact that tested arrays are too small.
It's a common optimization in sorting algorithms to switch to insertion sort for small arrays, and insertion sort is inherently stable. See #define SMALL_QUICKSORT 15 constant in numpy's code for instance.
More generally, it's common to switch strategy for small arrays in sorting algorithm, so it's good to test long enough arrays when testing sort/argsort.
Activity
[-]Sort-stability is not verified in argsort tests[/-][+]argsort: default value of stable not tested[/+]cakedev0 commentedon Oct 2, 2025
I would have been happy to write the PR, but the heavy use ofhypothesismakes it hard for me, I don't know this lib.Edit: I did the PR in the end: #390