changed attr accessors to attr readers to avoid accidental renaming