more work on setter