This will fail as `split_values` is an iterator, not a list and therefore the length is not known. I think you and Bevinsky have made lots of functions into generators, which is good general practice but in a few cases it may not make sense. Perhaps we can judiciously pick where we use lists out of necessity?