mendicant-original / mentoring

9 stars 1 forks source link

Request for mentoring session #38

Closed ghost closed 12 years ago

ghost commented 12 years ago

Code review of either the chess validator or six degrees of separation submissions.

ericgj commented 12 years ago

Hi Sherin, how about this Thursday (Dec 15) at 14:00 UTC?

Also, could you take your code down from those public repos? We try to keep solutions private for awhile, even after the admissions are closed. Thanks! I have a copy of the zip file you submitted.

ghost commented 12 years ago

Yes, this Thursday is good.

Public repositories have been removed.

ericgj commented 12 years ago

OK, great. We can meet on #rmu-study-hall-2 channel on freenode on Thursday.

ericgj commented 12 years ago

@csherin: my mistake - it's ok to release your solutions publicly after admissions closes. sorry for the hassle. I do have your zip file too, but if you want to release them publicly you can.

ghost commented 12 years ago

I pushed them to github for your benefit anyways. No problem :-)

ericgj commented 12 years ago

@csherin , great to chat with you today. I forgot to record the session, but I've pasted the transcript here.

Thinking about it some more, that board.neighbors_at doesn't quite work, you'd need to pass in either the piece or the square coordinates... in which case, it starts to point to the need for a Square object that maybe could mediate between the pieces and the board. So then you could have the pieces call square.neighbors_at and not deal with the board directly. Anyway... another idea.

ericgj commented 12 years ago

PS, to help improve the mentoring sessions, I'd really appreciate if you could fill out this survey.

ghost commented 12 years ago

@ericgj Filled out the survey. Thank you for the ideas and feedback.