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
Thoughts? Can you think of a better way? Should we allow the current method to be chosen (such as if seq is expected to be distinct nearly all the time)?
The text was updated successfully, but these errors were encountered:
Probably not a big deal in either way. I'd suggest either going with what you have now or using the old method if isinstance(seq, [list, tuple, set]). I'm ambivalent between the two.
eriknw
added a commit
to eriknw/toolz
that referenced
this issue
Nov 13, 2013
itertoolz.isdistinct
does not accept iterators, and it does not short-circuit. Both of these issues can be resolved as follows:Here are benchmarks when all elements are distinct:
Thoughts? Can you think of a better way? Should we allow the current method to be chosen (such as if
seq
is expected to be distinct nearly all the time)?The text was updated successfully, but these errors were encountered: