Do we need to change it to accessor?