Skip to content

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 of logger.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

Merge request reports