Skip to content

Commit

Permalink
Env: dict -> Mapping
Browse files Browse the repository at this point in the history
  • Loading branch information
MattToast committed Jun 26, 2024
1 parent 5572024 commit cabc2a0
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion smartsim/settings/builders/launch/alps.py
Original file line number Diff line number Diff line change
Expand Up @@ -220,7 +220,7 @@ def set(self, key: str, value: str | None) -> None:
self._launch_args[key] = value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return (
"aprun",
Expand Down
2 changes: 1 addition & 1 deletion smartsim/settings/builders/launch/dragon.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ def set(self, key: str, value: str | None) -> None:
self._launch_args[key] = value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> DragonRunRequest:
exe_, *args = exe.as_program_arguments()
return DragonRunRequest(
Expand Down
2 changes: 1 addition & 1 deletion smartsim/settings/builders/launch/local.py
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,6 @@ def set(self, key: str, value: str | None) -> None:
self._launch_args[key] = value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return exe.as_program_arguments()
2 changes: 1 addition & 1 deletion smartsim/settings/builders/launch/lsf.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ def set(self, key: str, value: str | None) -> None:
self._launch_args[key] = value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return (
"jsrun",
Expand Down
6 changes: 3 additions & 3 deletions smartsim/settings/builders/launch/mpi.py
Original file line number Diff line number Diff line change
Expand Up @@ -225,7 +225,7 @@ def launcher_str(self) -> str:
return LauncherType.Mpirun.value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return ("mpirun", *self.format_launch_args(), "--", *exe.as_program_arguments())

Expand All @@ -237,7 +237,7 @@ def launcher_str(self) -> str:
return LauncherType.Mpiexec.value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return (
"mpiexec",
Expand All @@ -254,7 +254,7 @@ def launcher_str(self) -> str:
return LauncherType.Orterun.value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return (
"orterun",
Expand Down
2 changes: 1 addition & 1 deletion smartsim/settings/builders/launch/pals.py
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ def set(self, key: str, value: str | None) -> None:
self._launch_args[key] = value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return (
"mpiexec",
Expand Down
2 changes: 1 addition & 1 deletion smartsim/settings/builders/launch/slurm.py
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ def set(self, key: str, value: str | None) -> None:
self._launch_args[key] = value

def finalize(
self, exe: ExecutableLike, env: dict[str, str | None]
self, exe: ExecutableLike, env: t.Mapping[str, str | None]
) -> t.Sequence[str]:
return (
"srun",
Expand Down
2 changes: 1 addition & 1 deletion smartsim/settings/builders/launchArgBuilder.py
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ def set(self, arg: str, val: str | None) -> None:
"""Set the launch arguments"""

@abstractmethod
def finalize(self, exe: ExecutableLike, env: dict[str, str | None]) -> _T:
def finalize(self, exe: ExecutableLike, env: t.Mapping[str, str | None]) -> _T:
"""Prepare an entity for launch using the built options"""

def format_launch_args(self) -> t.Union[t.List[str], None]:
Expand Down

0 comments on commit cabc2a0

Please sign in to comment.