I've added two blank lines (one before and after the multi-line function ```getIdHashPartition```).  Is our intent that "any two consecutive one-line variables or functions do not need a separating blank line; everything else does"?  Seems reasonable enough.