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

Collection.combinations() #747

Open
gavinking opened this issue Sep 15, 2015 · 6 comments
Open

Collection.combinations() #747

gavinking opened this issue Sep 15, 2015 · 6 comments
Labels
Milestone

Comments

@gavinking
Copy link
Member

To match @gdejohn's implementation of Collection.permutations(), we also need combinations().

@gavinking gavinking added this to the 1.2 milestone Sep 15, 2015
@FroMage
Copy link
Member

FroMage commented Sep 21, 2015

Is that really for 1.2?

@gdejohn
Copy link
Contributor

gdejohn commented Sep 21, 2015

It's a pretty small thing. I was hoping to find a bit of free time this week to work on it, assuming @gavinking isn't already working on it.

@gavinking
Copy link
Member Author

Im not, no

Sent from my iPhone

On 21 Sep 2015, at 18:08, Griffin DeJohn notifications@github.com wrote:

It's a pretty small thing. I was hoping to find a bit of free time this week to work on it, assuming @gavinking isn't already working on it.


Reply to this email directly or view it on GitHub.

@FroMage
Copy link
Member

FroMage commented Oct 14, 2015

Shall I move this to 1.3?

@gdejohn
Copy link
Contributor

gdejohn commented Oct 14, 2015

Yesterday I went to translate some Java code I had lying around for this, but I realized it assumed a set (no duplicate elements). I need to give it a bit more thought to avoid repeated combinations. I've been terribly busy with school, but I'll commit to figuring it out this weekend.

@FroMage
Copy link
Member

FroMage commented Oct 16, 2015

I'll move it to 1.3

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

No branches or pull requests

3 participants