This branch looks good. Here are a couple of things I spotted
while reading over it.
I believe that this bit is a syntax error in Python 2.6 as dict
comprehensions were added in 2.7. (line 132 of the diff, from
scrub_pofiletranslator.py):
return {poft.personID: poft for poft in pofts}
Converting it to something like this should work (untested):
return dict((poft.personID, poft) for poft in pofts)
Since you wrap all calls to measure_distance in abs, you could just put
the abs in measure_distance.
This branch looks good. Here are a couple of things I spotted
while reading over it.
I believe that this bit is a syntax error in Python 2.6 as dict nslator. py):
comprehensions were added in 2.7. (line 132 of the diff, from
scrub_pofiletra
return {poft.personID: poft for poft in pofts}
Converting it to something like this should work (untested):
return dict((poft. personID, poft) for poft in pofts)
Since you wrap all calls to measure_distance in abs, you could just put
the abs in measure_distance.