Skip to content

Commit aa6ab95

Browse files
author
Al Rigazzi
committed
Post-merge fixes
1 parent 146b31c commit aa6ab95

4 files changed

Lines changed: 7 additions & 219 deletions

File tree

conftest.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -351,7 +351,7 @@ def get_batch_resources() -> dict:
351351
def get_base_run_settings(
352352
exe: str, args: list[str], nodes: int = 1, ntasks: int = 1, **kwargs: t.Any
353353
) -> RunSettings:
354-
run_args: dict[str, int, str | float | None] = {}
354+
run_args: dict[str, int | str | float | None] = {}
355355

356356
if test_launcher == "slurm":
357357
run_args = {"--nodes": nodes, "--ntasks": ntasks, "--time": "00:10:00"}
@@ -394,7 +394,7 @@ def get_base_run_settings(
394394
def get_run_settings(
395395
exe: str, args: list[str], nodes: int = 1, ntasks: int = 1, **kwargs: t.Any
396396
) -> RunSettings:
397-
run_args: dict[str, int, str | float | None] = {}
397+
run_args: dict[str, int | str | float | None] = {}
398398

399399
if test_launcher == "slurm":
400400
run_args = {"nodes": nodes, "ntasks": ntasks, "time": "00:10:00"}

doc/changelog.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,11 @@ Detailed Notes
5555
`pathlib.Path` instances to provide a single source of truth for SmartSim's
5656
hidden workspace directories and Dragon launcher log locations.
5757
([SmartSim-PR789](https://github.com/CrayLabs/SmartSim/pull/789))
58+
- Copyright headers have been updated from "2021-2024" to "2021-2025" across
59+
271 files including Python source files, configuration files, documentation,
60+
tests, Docker files, shell scripts, and other supporting files to reflect the
61+
new year.
62+
([SmartSim-PR790](https://github.com/CrayLabs/SmartSim/pull/790))
5863
- Python 3.12 is now supported. TensorFlow 2.16.2 and PyTorch 2.7.1 library
5964
files are installed as part of `smart build` process when available. On Mac,
6065
ONNX runtime 1.22.0 is now installed, together with ONNX 1.16.

smartsim/_core/control/controller.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -478,12 +478,6 @@ def _launch_orchestrator(
478478

479479
# if the orchestrator was launched as a batch workload
480480
if orchestrator.batch:
481-
metadata_dir = (
482-
pathlib.Path(orchestrator.path)
483-
/ CONFIG.metadata_subdir
484-
/ "database"
485-
/ orchestrator.name
486-
)
487481
orc_batch_step, substeps = self._create_batch_job_step(
488482
orchestrator, metadata_dir
489483
)
@@ -496,12 +490,6 @@ def _launch_orchestrator(
496490

497491
# if orchestrator was run on existing allocation, locally, or in allocation
498492
else:
499-
metadata_dir = (
500-
pathlib.Path(orchestrator.path)
501-
/ CONFIG.metadata_subdir
502-
/ "database"
503-
/ orchestrator.name
504-
)
505493
db_steps = [
506494
(self._create_job_step(db, metadata_dir), db)
507495
for db in orchestrator.entities

tests/test_manifest_metadata_directories.py

Lines changed: 0 additions & 205 deletions
This file was deleted.

0 commit comments

Comments
 (0)