Joseph Reagle <reagle at mit.edu> writes: > inSRC = set([bio.name for bio in bios.values()]) You should use: inSRC = set(bio.name for bio in bios.values()) without the square brackets. That avoids creating an intermediate list.