should this have a default value?