Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this structure, but I think this function belongs in meshtastic.util, perhaps something like to_node_num there. (side note that most of the functions there are camelcase rather than underscores, but I think we may as well follow pep8 when we can).

Doesn't need to block merging, but if you're able to move this there that'd be appreciated!