change back to attr_accessor