This is really minor: I know we don't have this written in any coding guidelines or anything but we try to keep variable names camel-cased or underscore separated; could you change `numcolors` to `numColors` or `num_colors`.