Skip to content
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

Add relative_complement and symmetric_difference #89

Merged
merged 3 commits into from
May 10, 2019
Merged

Add relative_complement and symmetric_difference #89

merged 3 commits into from
May 10, 2019

Conversation

AljoschaMeyer
Copy link
Contributor

Closing #88, this adds methods for computing the relative complement. Also adds symmetric_difference methods, making difference an alias of the former.

I'd personally go further and deprecate difference. Reading code that says difference but actually computes symmetric difference sounds really confusing to me. But that's not my call to make :)

@bodil
Copy link
Owner

bodil commented May 10, 2019

I'd prefer to keep the alias, to be less confusing to people coming from Haskell and others.

Looks very good otherwise. 👍

@bodil bodil merged commit d2befa1 into bodil:master May 10, 2019
@abrassel
Copy link

As someone coming from a non-haskell / math background, difference being an alias for symmetric_difference was recently a pretty unpleasant surprise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants