Skip to content

fixing imports in univa_engine.py#3

Open
hmcgovern wants to merge 1 commit intoTomLippincott:masterfrom
hmcgovern:univa-imports
Open

fixing imports in univa_engine.py#3
hmcgovern wants to merge 1 commit intoTomLippincott:masterfrom
hmcgovern:univa-imports

Conversation

@hmcgovern
Copy link

When using steamroller for SLURM, I found that univa_engine.py is missing a few imports which prevent steamroller from using any engine other than local. This pull request:

  • adds import statements for os, logging, subprocess, and shlex, which are called in the script
  • comments out an unused line with reference to a variable grid_resources, which is not defined in the current version of the univa_engine.py script

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant