Bunch of Tiny Fixes and Enhancements

[mcfletch]

Hey there,

Thanks for all the work on the package. We've just spent an hour or two here getting everything connected and working on Linux/Ubuntu/Kubuntu 19.10 on a new Boost Vernie and thought we'd kick back the changes we made to get the demo running here.

Feel free to cherry pick individual commits if you'd rather skip some of the changes.

• The first change is to use setuptools for setup, as python3.6 distutils reused to honour the extras_require parameters
• second is to allow for debugging an error seen where an assert is raised for the incoming type of a message, but it simply bombed out the script without telling us what was wrong
• third is to deal with cases where there is truncated input on the input, so the struct unpacking was failing. The most common failure case we saw was to have 0 bytes coming in on a 1-byte unpack
• fourth is the most invasive change, which reworks the demo to use argparse and allow for providing a URL for the robot to use, and also to specify that a single demo be run rather than always running all demos (to make it easier to debug a single demo)

Take care,
Mike

[codecov-io]

Codecov Report

Merging #41 into master will increase coverage by 30.24%.
The diff coverage is 60%.

@@             Coverage Diff             @@
##           master      #41       +/-   ##
===========================================
+ Coverage   45.24%   75.49%   +30.24%     
===========================================
  Files          37       17       -20     
  Lines        3421     2057     -1364     
===========================================
+ Hits         1548     1553        +5     
+ Misses       1873      504     -1369

Impacted Files	Coverage Δ
setup.py	0% <0%> (ø)	⬆️
pylgbst/messages.py	75.8% <50%> (-0.17%)	⬇️
pylgbst/utilities.py	88.46% <72.72%> (-6.28%)	⬇️
examples/bb8joystick/program.py
examples/plotter/__init__.py
examples/bb8joystick/__init__.py
examples/harmonograph/__init__.py
examples/automata/__init__.py
examples/plotter/try.py
examples/vernie/run_away_game.py
... and 12 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7dc8b80...196761e. Read the comment docs.

[undera]

It's good to add some message to it, like 'Unexpected type message type: %s'. Let's do it good if we got to improving it :)

[undera]

This code needs some formatting to comply to PEP-9.

[undera]

don't comment, just delete

[mcfletch]

Sure, if you're okay with the change I'll commit to it.

[undera]

Please do regular formatting, like standards require

[mcfletch]

Didn't realise you had a formatting spec. I don't see one in the repository, so I'm assuming black, though that seems to be producing rather large diffs on the modules to which I'm applying it.

[undera]

Thanks for contributing. I've put some comments, please address them.
The most intrusive change is the one with check_unpack - I totally don't expect this to happen on a regular basis, so I don't fully like this change. I'd love to understand why can this happen and if there is more aesthetic way to deal with the situation.