search

LambdaConglomerate / x9115lam

2 stars 0 forks source link

Review Questions #16

Closed meneal closed 8 years ago

meneal commented 8 years ago

I'm just opening up an issue for this. Kind of thinking it might make sense to work together in getting through these. At least to come up with some sort of communal answer to them. I'll start posting my answers this afternoon as I go through the questions.

ghost commented 8 years ago

Sounds good to me. I have most of the first two done already, and I'm working on the next two for today.

meneal commented 8 years ago

How much time are they taking?

On Fri, Oct 9, 2015 at 2:20 PM, Joseph Sankar notifications@github.com wrote:

Sounds good to me. I have most of the first two done already, and I'm working on the next two for today.

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

ghost commented 8 years ago

I'd say 45 minutes to an hour each. I had to look up a lot of definitions on the first one. The second one didn't take as long, though I did have to look up some syntax.

ghost commented 8 years ago

I just pushed what I have for the first three reviews. I know the second one isn't finished.

If there is something you disagree with, please let me know. Also, feel free to add any additional information you want. If it makes it clearer, we can put our initials beside each of our responses.

I'm hoping to have the second one and fourth one finished by the end of tonight.

meneal commented 8 years ago

Thanks for doing that! I'll be working on this some more this evening. I've been really concentrating on HW6, but I really need to get on this too.

meneal commented 8 years ago

There was a mistake in REVIEW2.md. I just pushed a change. I also pushed a runnable code block to show how it fails and a way to make it work.

ghost commented 8 years ago

Looks like I misread the question. I thought it meant print lines that are longer than 20 characters, but he really just wants there to be more than 20 lines instead. I'm assuming there is no restriction on if blank lines are counted through. But good catch @meneal!

meneal commented 8 years ago

Do you guys care if we just merge the two versions of REVIEW4?

meneal commented 8 years ago

I've added some stuff as I went to the review questions and fixed stuff when I thought it was broken. Hope you guys don't mind.

meneal commented 8 years ago

I added review 7 up there. Add whatever you want to it.

aisobran commented 8 years ago

For Review 2 question 2c. You guys are both right.

Every function without a return, returns None. The function doesn't necessarily break, you cant call it with += syntactic sugar but it still works as intended if you use iadd. Commenting it out disables method chaining but I suppose that's obvious.

meneal commented 8 years ago

Oh shit. I actually misread that. I thought it meant what would happen if you completely deleted the iadd funciton!

On Mon, Oct 12, 2015 at 8:07 PM, Alexander Sobran notifications@github.com wrote:

For Review 2 question 2c. You guys are both right.

Every function without a return, returns None. The function doesn't necessarily break, you cant call it with += syntactic sugar but it still works as intended if you use iadd. Commenting it out also gets disables method chaining but I suppose that's obvious.

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

aisobran commented 8 years ago

Either way you're right about += not working because it's self-referential

ghost commented 8 years ago

Just a heads up, I made a slight change to Review6 where we had to label A and B in the figure. I understood it as what term best describes A and B (precision and recall). Let me know if you disagree.

aisobran commented 8 years ago

@WumpaFruit Yeah that's right.

Do you guys want to have a study session from 3:30 - 5:20 in the library?

meneal commented 8 years ago

I'm into that. Should we reserve a room?

On Tue, Oct 13, 2015 at 8:07 AM, Alexander Sobran notifications@github.com wrote:

@WumpaFruit https://github.com/WumpaFruit Yeah that's right.

Do you guys want to have a study session from 3:30 - 5:20 in the library?

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

aisobran commented 8 years ago

I have 4322 reserved from 3:30 - 5:30

ghost commented 8 years ago

I'll be there at 4. On Oct 13, 2015 10:11 AM, "Alexander Sobran" notifications@github.com wrote:

I have 4322 reserved from 3:30 - 5:30

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

meneal commented 8 years ago

I'm up there now. Fire drill is over.

On Tuesday, October 13, 2015, Joseph Sankar notifications@github.com wrote:

I'll be there at 4. On Oct 13, 2015 10:11 AM, "Alexander Sobran" <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

I have 4322 reserved from 3:30 - 5:30

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

.

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

ghost commented 8 years ago

Okay, I'm heading there too. Will be there in a few.

On Tue, Oct 13, 2015 at 3:45 PM, Matthew Neal notifications@github.com wrote:

I'm up there now. Fire drill is over.

On Tuesday, October 13, 2015, Joseph Sankar notifications@github.com wrote:

I'll be there at 4. On Oct 13, 2015 10:11 AM, "Alexander Sobran" <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

I have 4322 reserved from 3:30 - 5:30

— Reply to this email directly or view it on GitHub <

https://github.com/LambdaConglomerate/x9115lam/issues/16#issuecomment-147726374

.

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

.

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

Joseph Sankar (About Me) http://josephsankar.me Master of Computer Science, Class of 2016, North Carolina State University Bachelor's in Computer Science, Class of 2015, North Carolina State University

ghost commented 8 years ago

My card's not working on the door. Are you all in there? On Oct 13, 2015 3:46 PM, "Joseph Sankar" jesankar@ncsu.edu wrote:

Okay, I'm heading there too. Will be there in a few.

On Tue, Oct 13, 2015 at 3:45 PM, Matthew Neal notifications@github.com wrote:

I'm up there now. Fire drill is over.

On Tuesday, October 13, 2015, Joseph Sankar notifications@github.com wrote:

I'll be there at 4. On Oct 13, 2015 10:11 AM, "Alexander Sobran" <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

I have 4322 reserved from 3:30 - 5:30

— Reply to this email directly or view it on GitHub <

https://github.com/LambdaConglomerate/x9115lam/issues/16#issuecomment-147726374

.

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

.

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

Joseph Sankar (About Me) http://josephsankar.me Master of Computer Science, Class of 2016, North Carolina State University Bachelor's in Computer Science, Class of 2015, North Carolina State University

ghost commented 8 years ago

Closing this.