PyRival
PyRival copied to clipboard
The new SortedList method naming scheme is not user friendly
The old method names were better as they directly copied sortedcontainers. Using multiset method names is annoying.
Can you be more specific?
SortedList in python uses bisect_left (not lower_bound) and bisect_right (not upper_bound) and add method (not insert) lower_bound and upper_bound is the method names for C++ and many python users arent very good at it, which makes it very confusing
lower_bound -> bisect_left upper_bound -> bisect_right insert -> add
ensure consistency between the main SortedList implementation and this implementation
A remove method would be good as s.pop(s.lower_bound(val))
is not so obvious