make these modes an enum please