Major rehash of parsing code

parsePGN has had a large chunk of code cut out, all the logic that tried to estimate the presence of an alias. On reflection this is too difficult and can easily be handled downstream. Also added other checks to catch names with bad characters, and some code to auto recognize CPUs.

DBI test has been deleted.

settings.yaml has been copied to the score PGN section.

The dockerfile now includes vi- this will probably change soon- as was just for testing.

run-client has been moved to the scorePGN direcory. The aliases field has been removed from the SQL.
Removed vim install from Dockerfile