why do we want an empty string as a default? I presume without it, it is a None?