Should other methods like on_tx_request have the '(normally overridden by inheriting class)' note?