I agree that it's only called here but I find it much more readable (you could even add a docstring saying at this private method sets self._reject_time and self._channel_type_idx). Can you please revert it?