should this be a val?