ideally, group_colours would be a named vector! I think at the moment it uses the colours alphabetically (like the factor levels by default).