didn't get that logic quite right