shouldn't this be a constant?