could you avoid abbreviating ``string`` to ``str`` here ? It does not add much value IMO