possibly check they are positive