Skip to content

Commit

Permalink
removed requirements that no longer apply to all backends
Browse files Browse the repository at this point in the history
  • Loading branch information
FrederickDeny committed Dec 28, 2023
1 parent 316b3a7 commit 62336c9
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 3 deletions.
2 changes: 0 additions & 2 deletions e4s_cl/cf/containers/barebones.py
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,6 @@ def _has_nvidia(self):

def run(self, command: List[str], overload: bool = True) -> int:
executable = self._executable()
if executable is None:
raise BackendNotAvailableError(self.__class__.__name__)

container_cmd = [executable, *self._prepare(command, overload)]

Expand Down
1 change: 0 additions & 1 deletion e4s_cl/cli/commands/__execute.py
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,6 @@ def _construct_parser(self):
parser.add_argument('--image',
type=str,
dest='image',
required=True,
help="Container image to use",
metavar='image')

Expand Down
2 changes: 2 additions & 0 deletions e4s_cl/cli/commands/launch.py
Original file line number Diff line number Diff line change
Expand Up @@ -297,6 +297,8 @@ def main(self, argv):
# Ensure the minimum fields required for launch are present
for _field in ['backend', 'image']:
if not getattr(parameters, _field, None):
if getattr(parameters, 'backend',None) == 'barebones':
continue
self.parser.error(
f"Missing field: '{_field}'. Specify it using the "
"appropriate option or by selecting a profile.")
Expand Down

0 comments on commit 62336c9

Please sign in to comment.