don't need the attr_accessor