Resolve "newMotorPositions variable not defined in runDozorm2"
Closes #5 (closed)
- Fix several paths that lead to undefined local variables. Not sure it is correct since nothing helps me judge that.
- When using python logging it is common to do
logger.info("... %s ...", somevar)
instead oflogger.info(f"... {somevar} ...")
. The reason is that you want to evaluate the string only when needed (i.e. if there are handlers and the log level is reached). - runDozorm2 called a function which uses
tempfile.mkstemp
without context manager. I fixed three such occurrences in the same module + unit tests to check that I did not change the behavior.
Edited by Wout De Nolf