this should be an attribute