some updates to impl section