more work on person class