search

LambdaConglomerate / x9115lam

2 stars 0 forks source link

Code 3 #6

Closed ghost closed 8 years ago

ghost commented 8 years ago

Let's talk about how we are going to split Code3. It looks like Sasha is working on 10.15 Exercise 8, so I will start working on the Employee class.

ghost commented 8 years ago

I just pushed the Employee class. Have a look and let me know how it looks.

It looks like poker will be a big one. It looks like we can divide the task according to 3-6 in 18.12 Exercise 6. Since Employee was relatively easy, I'm fine with starting poker. Let me know if you have any objections to this.

ghost commented 8 years ago

I just pushed the checks for poker hand methods. If anyone wants to work on the classify method, let me know. Otherwise, I'll start working on it tomorrow night.

meneal commented 8 years ago

I definitely want to work on some of this. I probably can't get working on it until Friday night or Sunday. Make sure to leave me with something to do. I don't really care how hard it is. I just haven't had a chance to get a good look at it yet. Will definitely look at the paper too and submit on Friday/Sunday.

On Thu, Sep 3, 2015 at 9:11 PM, Joseph Sankar notifications@github.com wrote:

I just pushed the checks for poker hand methods. If anyone wants to work on the classify method, let me know. Otherwise, I'll start working on it tomorrow night.

— Reply to this email directly or view it on GitHub https://github.com/LambdaConglomerate/x9115lam/issues/6#issuecomment-137615278 .

aisobran commented 8 years ago

10.8 is done and the pic is added.

meneal commented 8 years ago

I'm working on classify now.

meneal commented 8 years ago

I think we need some tests for this. Would you knock that out Sasha? :)

meneal commented 8 years ago

I think poker hands is pretty much complete now. Running a test with 10 million decks on AWS just to see if I can get it to output the same probabilities as on Wikipedia. I'll post the output whenever the test finishes.

meneal commented 8 years ago

Output is posted in the README. I think this is done but somebody else should check over it and either submit or state what else needs to be done.

ghost commented 8 years ago

I ran the numbers on my machine and they are very similar to yours and to the ones on Wikipedia. I am okay with submitting if Sasha approves.

aisobran commented 8 years ago

Good to go.

meneal commented 8 years ago

Make sure if you close to put in the tiny url for the submission. I just put this one in.

On Tue, Sep 8, 2015 at 4:29 PM, Alexander Sobran notifications@github.com wrote:

Closed #6 https://github.com/LambdaConglomerate/x9115lam/issues/6.

— Reply to this email directly or view it on GitHub https://github.com/LambdaConglomerate/x9115lam/issues/6#event-404105814.