Rings the bells and spread the word, vacationing family has finally vacated! I am now free from the influence of 4 adults and 5 dogs intruding on my workplace. I have to say I really expected things to go a little better than they did; Originally half of the family was staying at a local campground, but when their time ran up and they were going to leave they found out their trailer was massively out of alignment. They needed to stay an extra 1/2 week for their alignment appointment, and as a result instead of only having 2 people/dogs at the house we had double. I don’t hate my family or anything, its just too many people and dogs in one place.
As for work things have been delayed but not so much I am behind schedule. It was very exciting to learn that many of the AI ‘fellows’ would be working on a Risk AI; It has forced me to think more deeply about how Risk works from an outsiders perspective (and probably going to improve the quality of my screencasts.) I should be recording my screencast’s video this week, and dubbing this week or next. I also received some positive feedback from my mentor Tyler about my map import code; Surprisingly (since I still undervalue myself as a coder) I was not far from the mark. If I put in a concentrated effort it should only be 1-2 days to really clean up that code. I also received some feedback about comments in my code from Mithro: I’m not terribly far from the correct path, but I will be “doxygenating” my functions most likely in the “wrap-up” week that is the last week of GSoC.
Another exciting development from Mithro was the creation of a “Wormholes” patch. The patch updates my Risk code to properly draw lines between planets. This development has to be one of the biggest things to happen to Risk this summer. Map imports was big, and cool, but “Line drawing” is a huge usability feature that Risk was sorely missing. A big thanks to Mithro and anyone else who aided him (and by proxy, me) in getting Risk to display properly.

mithro | 28-Jul-08 at 8:35 pm | Permalink
BTW My patch does not touch the map import code. You will have to update that to create wormholes too. (As I mentioned in my email you must be running git tpclient-pywx to see the results.)
Don’t leave comments till the last moment. You will find that adding them will help you write cleaner and better code. (And hence help with you clean up attempts.)
Don’t forget that “magic number” are bad even if well commented. (Use constants instead.)