Clean up the `solution.read` function

PR does:
 - Addresses #404 (docstring error).  
 - Adds a `ValueError` exception for a non-standard `file_format` as this is a much better way to inform the user that something has gone wrong rather than the alternative where this raises a non-informative exception.
 - PEP 8 compliance