should this be an ``elif``? it isn't clear how this branch relates to the ``if``