We wanted to be clear we expect the field to be a ModelType.

Do you think it is clearer to call the iterator variable `model_type` or `field`?