I don't think you need the `self.` for the data names here.