-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
ENH: Add sort parameter to other set operations if possible #24471
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
yes sort=True (default for compat) should be added to all set operations |
@fdroessler As discussed, if you see any other things we need to do to make the implementation of the |
It seems that sort=True hasn't been implemented in the recent release version. I tested and got "ValueError: The 'sort' keyword only takes the values of None or False; True was passed." I can try on this issue. |
The set ops all have sort keyword. Closing as complete. |
As part of #22811 I added a
sort
parameter to thedifference
set operation with a default ofTrue
. The other set operations do not have this parameter but some of them sort by default if possible. Should we add asort
parameter to the other set operations too if possible for consistency?TODO:
sort
option. TimedeltaIndex.intersection has nosort
option. #24966sort
parameter forRangeIndex.union
(ENH: Add sort parameter to RangeIndex.union (#24471) #25788)sort=True
and deprecatesort=None
in future release?The text was updated successfully, but these errors were encountered: