Consider returning subprocess.CompletedProcess instance from Engine1D.run()

[lizaclark]

It would be cool if Engine1D.run() returned the subprocess.CompletedProcess instance that is returned from the subprocess.run() call at

mikepluspy/mikeplus/engines/engine1d.py

Line 49 in 69d0bdd

subprocess.run([mike1d_exec, str(dbOrMuppFile), f"-simulationid={simMuid}", f"-logfilename={log_file}"])

. This would allow users to check for subprocess errors within a script, if running a loop for instance.

https://docs.python.org/3/library/subprocess.html#subprocess.CompletedProcess

[ryan-kipawa]

Sounds like a really great suggestion @lizaclark, thanks! Would you be interested in submitting a pull request with this change?