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

Comparison with other types #19

Conversation

kaplun
Copy link
Member

@kaplun kaplun commented Dec 8, 2014

Signed-off-by: Samuele Kaplun samuele.kaplun@cern.ch
Reported-by: Dimitrios Semitsoglou-Tsiapos dsemitso@cern.ch

@kaplun kaplun added this to the v2.2 milestone Dec 8, 2014
@kaplun kaplun force-pushed the 18-comparison-with-other-types branch from 7430b6a to 3d95c35 Compare December 8, 2014 21:32
@kaplun
Copy link
Member Author

kaplun commented Dec 8, 2014

Note @dset0x, this branch does not fully mimic the behavior of set because it is returning NotImplemented as soon as one of the two elements being compared is not an intbitset. Apparently Python set is instead casting to a set the rhs element in a comparison.

* Comparing an intbitset with anything that is not an intbitset
  should behave as a set would. E.g.
  intbitset([1,2,3]) == set([1,2,3]) is False.
  (closes inveniosoftware-contrib#18)

Signed-off-by: Samuele Kaplun <samuele.kaplun@cern.ch>
Reported-by: Dimitrios Semitsoglou-Tsiapos <dsemitso@cern.ch>
@kaplun kaplun force-pushed the 18-comparison-with-other-types branch from 3d95c35 to 3fb6d9b Compare July 13, 2015 08:24
@kaplun kaplun modified the milestones: v2.3, v2.2 Sep 4, 2015
@pombredanne
Copy link
Collaborator

I will be reviewing and may be rebasing and merging this at last!

@dset0x
Copy link

dset0x commented Dec 23, 2019

Four years sure fly by quickly.

@lnielsen
Copy link
Contributor

Closing PR as it's stale for several years.

@lnielsen lnielsen closed this Feb 16, 2022
@pombredanne
Copy link
Collaborator

@lnielsen the code is stable and this PR is still valid... so let me review and merge. It is LTGM otherwise.

@pombredanne pombredanne reopened this Feb 17, 2022
@pombredanne
Copy link
Collaborator

pombredanne commented Feb 21, 2022

Actually this is not readily mergeable... my bad. Closing indeed.

@dset0x
Copy link

dset0x commented Feb 21, 2022

I was really counting on this one. (´д`)

@lnielsen
Copy link
Contributor

Just FYI, I was doing cleanup of old pull requests in inveniosoftware organisation, that was the main reason to close it since it was open since 2014 :-)

@pombredanne
Copy link
Collaborator

@dset0x we can implement it still... but the code in this PR was stale and not exactly working as expected.

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

Successfully merging this pull request may close these issues.

Support comparisons with other types.
4 participants