shepherd-core 2025.6.3__py3-none-any.whl → 2025.8.1__py3-none-any.whl

shepherd_core/data_models/task/firmware_mod.py

@@ -4,9 +4,7 @@ from collections.abc import Set as AbstractSet
 from pathlib import Path
 from pathlib import PurePosixPath
 from typing import Annotated
-from typing import Optional
 from typing import TypedDict
-from typing import Union
 
 from pydantic import Field
 from pydantic import model_validator
@@ -31,9 +29,9 @@ from .helper_paths import path_posix
 class FirmwareModTask(ShpModel):
     """Config for Task that adds the custom ID to the firmware & stores it into a file."""
 
-    data: Union[FirmwareStr, Path]
+    data: FirmwareStr | Path
     data_type: FirmwareDType
-    custom_id: Optional[IdInt16] = None
+    custom_id: IdInt16 | None = None
     firmware_file: Path
 
     verbose: Annotated[int, Field(ge=0, le=4)] = 2
@@ -95,6 +93,7 @@ class FirmwareModTask(ShpModel):
         return cls(**kwargs)
 
     def is_contained(self, paths: AbstractSet[PurePosixPath]) -> bool:
+        """Limit paths to allowed directories."""
         all_ok = any(self.firmware_file.is_relative_to(path) for path in paths)
         if isinstance(self.data, Path):
             all_ok = any(self.data.is_relative_to(path) for path in paths)

shepherd_core/data_models/task/harvest.py

@@ -6,12 +6,10 @@ from datetime import timedelta
 from pathlib import Path
 from pathlib import PurePosixPath
 from typing import Annotated
-from typing import Optional
 
 from pydantic import Field
 from pydantic import model_validator
 from typing_extensions import Self
-from typing_extensions import deprecated
 
 from shepherd_core.data_models.base.shepherd import ShpModel
 from shepherd_core.data_models.base.timezone import local_tz
@@ -35,14 +33,13 @@ class HarvestTask(ShpModel):
     """
     force_overwrite: bool = False
     """ ⤷ Overwrite existing file"""
-    output_compression: Optional[Compression] = Compression.default
+    output_compression: Compression | None = Compression.default
     """ ⤷ should be 1 (level 1 gzip), lzf, or None (order of recommendation)"""
 
-    time_start: Optional[datetime] = None
+    time_start: datetime | None = None
     """ timestamp or unix epoch time, None = ASAP"""
-    duration: Optional[timedelta] = None
+    duration: timedelta | None = None
     """ ⤷ Duration of recording in seconds, None = till EOFSys"""
-    abort_on_error: Annotated[bool, deprecated("has no effect")] = False
 
     # emulation-specific
     use_cal_default: bool = False
@@ -52,7 +49,7 @@ class HarvestTask(ShpModel):
     """ ⤷ Choose one of the predefined virtual harvesters or configure a new one
     """
     power_tracing: PowerTracing = PowerTracing()
-    sys_logging: Optional[SystemLogging] = SystemLogging()
+    sys_logging: SystemLogging | None = SystemLogging()
 
     verbose: Annotated[int, Field(ge=0, le=4)] = 2
     """ ⤷ 0=Errors, 1=Warnings, 2=Info, 3=Debug"""
@@ -75,12 +72,11 @@ class HarvestTask(ShpModel):
     @model_validator(mode="after")
     def post_validation(self) -> Self:
         # TODO: limit paths
-        has_time = self.time_start is not None
+        has_time = False  # TODO: deactivated, self.time_start is not None
         time_now = datetime.now().astimezone()
         if has_time and self.time_start < time_now:
             msg = (
-                "Start-Time for Emulation can't be in the past "
-                f"('{self.time_start}' vs '{time_now}'."
+                f"Start-Time for Harvest can't be in the past ('{self.time_start}' vs '{time_now}'."
             )
             raise ValueError(msg)
         if self.duration and self.duration.total_seconds() < 0:
@@ -88,4 +84,5 @@ class HarvestTask(ShpModel):
         return self
 
     def is_contained(self, paths: AbstractSet[PurePosixPath]) -> bool:
+        """Limit paths to allowed directories."""
         return any(self.output_path.is_relative_to(path) for path in paths)

shepherd_core/data_models/task/observer_tasks.py

@@ -5,7 +5,6 @@ from datetime import datetime
 from pathlib import Path
 from pathlib import PurePosixPath
 from typing import Annotated
-from typing import Optional
 
 from pydantic import validate_call
 from typing_extensions import Self
@@ -29,20 +28,20 @@ class ObserverTasks(ShpModel):
     observer: NameStr
 
     # PRE PROCESS
-    time_prep: Optional[datetime] = None  # TODO: currently not used
+    time_prep: datetime | None = None  # TODO: currently not used
     root_path: Path
 
     # fw mod, store as hex-file and program
-    fw1_mod: Optional[FirmwareModTask] = None
-    fw2_mod: Optional[FirmwareModTask] = None
-    fw1_prog: Optional[ProgrammingTask] = None
-    fw2_prog: Optional[ProgrammingTask] = None
+    fw1_mod: FirmwareModTask | None = None
+    fw2_mod: FirmwareModTask | None = None
+    fw1_prog: ProgrammingTask | None = None
+    fw2_prog: ProgrammingTask | None = None
 
     # MAIN PROCESS
-    emulation: Optional[EmulationTask] = None
+    emulation: EmulationTask | None = None
 
     # deprecations, TODO: remove before public release
-    owner_id: Annotated[Optional[IdInt], deprecated("not needed anymore")] = None
+    owner_id: Annotated[IdInt | None, deprecated("not needed anymore")] = None
     abort_on_error: Annotated[bool, deprecated("has no effect")] = False
 
     # post_copy / cleanup, Todo: could also just intake emuTask
@@ -53,12 +52,14 @@ class ObserverTasks(ShpModel):
 
     @classmethod
     @validate_call
-    def from_xp(cls, xp: Experiment, tb: Testbed, tgt_id: IdInt) -> Self:
+    def from_xp(cls, xp: Experiment, xp_folder: str | None, tb: Testbed, tgt_id: IdInt) -> Self:
         if not tb.shared_storage:
             raise ValueError("Implementation currently relies on shared storage!")
 
         obs = tb.get_observer(tgt_id)
-        root_path = tb.data_on_observer / "experiments" / xp.folder_name()
+        if xp_folder is None:
+            xp_folder = xp.folder_name()  # moved a layer up for consistent naming
+        root_path = tb.data_on_observer / "experiments" / xp_folder
         fw_paths = [root_path / f"fw{_i}_{obs.name}.hex" for _i in [1, 2]]
 
         return cls(
@@ -94,6 +95,7 @@ class ObserverTasks(ShpModel):
         return values
 
     def is_contained(self, paths: AbstractSet[PurePosixPath]) -> bool:
+        """Limit paths to allowed directories."""
         all_ok = any(self.root_path.is_relative_to(path) for path in paths)
         all_ok &= self.fw1_mod.is_contained(paths)
         all_ok &= self.fw2_mod.is_contained(paths)

shepherd_core/data_models/task/programming.py

@@ -4,7 +4,6 @@ from collections.abc import Set as AbstractSet
 from pathlib import Path
 from pathlib import PurePosixPath
 from typing import Annotated
-from typing import Optional
 
 from pydantic import Field
 from pydantic import model_validator
@@ -60,7 +59,7 @@ class ProgrammingTask(ShpModel):
         tgt_id: IdInt,
         mcu_port: MCUPort,
         fw_path: Path,
-    ) -> Optional[Self]:
+    ) -> Self | None:
         obs = tb.get_observer(tgt_id)
         tgt_cfg = xp.get_target_config(tgt_id)
 
@@ -79,4 +78,5 @@ class ProgrammingTask(ShpModel):
         )
 
     def is_contained(self, paths: AbstractSet[PurePosixPath]) -> bool:
+        """Limit paths to allowed directories."""
         return any(self.firmware_file.is_relative_to(path) for path in paths)

shepherd_core/data_models/task/testbed_tasks.py

@@ -4,14 +4,11 @@ from pathlib import Path
 from pathlib import PurePosixPath
 from typing import TYPE_CHECKING
 from typing import Annotated
-from typing import Optional
 
 from pydantic import Field
 from pydantic import validate_call
 from typing_extensions import Self
-from typing_extensions import deprecated
 
-from shepherd_core.data_models.base.content import IdInt
 from shepherd_core.data_models.base.content import NameStr
 from shepherd_core.data_models.base.shepherd import ShpModel
 from shepherd_core.data_models.experiment.experiment import Experiment
@@ -29,30 +26,30 @@ class TestbedTasks(ShpModel):
     name: NameStr
     observer_tasks: Annotated[list[ObserverTasks], Field(min_length=1, max_length=128)]
 
-    # deprecated, TODO: remove before public release
-    email_results: Annotated[Optional[bool], deprecated("not needed anymore")] = False
-    owner_id: Annotated[Optional[IdInt], deprecated("not needed anymore")] = None
-
     @classmethod
     @validate_call
-    def from_xp(cls, xp: Experiment, tb: Optional[Testbed] = None) -> Self:
+    def from_xp(cls, xp: Experiment, tb: Testbed | None = None) -> Self:
         if tb is None:
             # TODO: is tb-argument really needed? prob. not
             tb = Testbed()  # this will query the first (and only) entry of client
 
         tgt_ids = xp.get_target_ids()
-        obs_tasks = [ObserverTasks.from_xp(xp, tb, _id) for _id in tgt_ids]
+        xp_folder = xp.folder_name()
+        obs_tasks = [ObserverTasks.from_xp(xp, xp_folder, tb, _id) for _id in tgt_ids]
         return cls(
             name=xp.name,
             observer_tasks=obs_tasks,
         )
 
-    def get_observer_tasks(self, observer: str) -> Optional[ObserverTasks]:
+    def get_observer_tasks(self, observer: str) -> ObserverTasks | None:
         for tasks in self.observer_tasks:
             if observer == tasks.observer:
                 return tasks
         return None
 
+    def get_observers(self) -> set[str]:
+        return {tasks.observer for tasks in self.observer_tasks}
+
     def get_output_paths(self) -> dict[str, Path]:
         # TODO: computed field preferred, but they don't work here, as
         #  - they are always stored in yaml despite "repr=False"
@@ -63,6 +60,7 @@ class TestbedTasks(ShpModel):
         return values
 
     def is_contained(self) -> bool:
+        """Limit paths to allowed directories."""
         paths_allowed: AbstractSet[PurePosixPath] = {
             PurePosixPath("/var/shepherd/"),
             PurePosixPath("/tmp/"),  # noqa: S108

shepherd_core/data_models/testbed/cape.py

@@ -4,8 +4,6 @@ from datetime import date
 from datetime import datetime
 from enum import Enum
 from typing import Any
-from typing import Optional
-from typing import Union
 
 from pydantic import Field
 from pydantic import model_validator
@@ -31,12 +29,12 @@ class Cape(ShpModel, title="Shepherd-Cape"):
     name: NameStr
     version: NameStr
     description: SafeStr
-    comment: Optional[SafeStr] = None
+    comment: SafeStr | None = None
     # TODO: wake_interval, calibration
 
     active: bool = True
-    created: Union[date, datetime] = Field(default_factory=datetime.now)
-    calibrated: Union[date, datetime, None] = None
+    created: date | datetime = Field(default_factory=datetime.now)
+    calibrated: date | datetime | None = None
 
     def __str__(self) -> str:
         return self.name

shepherd_core/data_models/testbed/gpio.py

@@ -3,7 +3,6 @@
 from enum import Enum
 from typing import Annotated
 from typing import Any
-from typing import Optional
 
 from pydantic import Field
 from pydantic import StringConstraints
@@ -30,16 +29,16 @@ class GPIO(ShpModel, title="GPIO of Observer Node"):
 
     id: IdInt
     name: NameStr
-    description: Optional[SafeStr] = None
-    comment: Optional[SafeStr] = None
+    description: SafeStr | None = None
+    comment: SafeStr | None = None
 
     direction: Direction = Direction.Input
-    dir_switch: Optional[Annotated[str, StringConstraints(max_length=32)]] = None
+    dir_switch: Annotated[str, StringConstraints(max_length=32)] | None = None
 
-    reg_pru: Optional[Annotated[str, StringConstraints(max_length=10)]] = None
-    pin_pru: Optional[Annotated[str, StringConstraints(max_length=10)]] = None
-    reg_sys: Optional[Annotated[int, Field(ge=0)]] = None
-    pin_sys: Optional[Annotated[str, StringConstraints(max_length=10)]] = None
+    reg_pru: Annotated[str, StringConstraints(max_length=10)] | None = None
+    pin_pru: Annotated[str, StringConstraints(max_length=10)] | None = None
+    reg_sys: Annotated[int, Field(ge=0)] | None = None
+    pin_sys: Annotated[str, StringConstraints(max_length=10)] | None = None
 
     def __str__(self) -> str:
         return self.name

shepherd_core/data_models/testbed/mcu.py

@@ -3,7 +3,6 @@
 from enum import Enum
 from typing import Annotated
 from typing import Any
-from typing import Optional
 
 from pydantic import Field
 from pydantic import model_validator
@@ -30,7 +29,7 @@ class MCU(ShpModel, title="Microcontroller of the Target Node"):
     id: IdInt
     name: NameStr
     description: SafeStr
-    comment: Optional[SafeStr] = None
+    comment: SafeStr | None = None
 
     platform: NameStr
     core: NameStr

shepherd_core/data_models/testbed/observer.py

@@ -3,7 +3,6 @@
 from datetime import datetime
 from typing import Annotated
 from typing import Any
-from typing import Optional
 
 from pydantic import Field
 from pydantic import IPvAnyAddress
@@ -33,7 +32,7 @@ class Observer(ShpModel, title="Shepherd-Sheep"):
     id: IdInt
     name: NameStr
     description: SafeStr
-    comment: Optional[SafeStr] = None
+    comment: SafeStr | None = None
 
     ip: IPvAnyAddress
     mac: MACStr
@@ -46,12 +45,12 @@ class Observer(ShpModel, title="Shepherd-Sheep"):
     """ ⤷ cfaed-floor"""
 
     active: bool = True
-    cape: Optional[Cape] = None
-    target_a: Optional[Target] = None
-    target_b: Optional[Target] = None
+    cape: Cape | None = None
+    target_a: Target | None = None
+    target_b: Target | None = None
 
     created: datetime = Field(default_factory=datetime.now)
-    alive_last: Optional[datetime] = None
+    alive_last: datetime | None = None
 
     def __str__(self) -> str:
         return self.name

shepherd_core/data_models/testbed/target.py

@@ -3,8 +3,6 @@
 from datetime import datetime
 from typing import Annotated
 from typing import Any
-from typing import Optional
-from typing import Union
 
 from pydantic import Field
 from pydantic import model_validator
@@ -30,15 +28,15 @@ class Target(ShpModel, title="Target Node (DuT)"):
     version: NameStr
     description: SafeStr
 
-    comment: Optional[SafeStr] = None
+    comment: SafeStr | None = None
 
     active: bool = True
     created: datetime = Field(default_factory=datetime.now)
 
-    testbed_id: Optional[IdInt16] = None
+    testbed_id: IdInt16 | None = None
     """ ⤷ is derived from ID (targets are still selected by id!)"""
-    mcu1: Union[MCU, NameStr]
-    mcu2: Union[MCU, NameStr, None] = None
+    mcu1: MCU | NameStr
+    mcu2: MCU | NameStr | None = None
 
     # TODO: programming pins per mcu should be here (or better in Cape)
 

shepherd_core/data_models/testbed/testbed.py

@@ -4,7 +4,6 @@ from datetime import timedelta
 from pathlib import Path
 from typing import Annotated
 from typing import Any
-from typing import Optional
 
 from pydantic import Field
 from pydantic import HttpUrl
@@ -29,9 +28,9 @@ class Testbed(ShpModel):
     id: IdInt
     name: NameStr
     description: SafeStr
-    comment: Optional[SafeStr] = None
+    comment: SafeStr | None = None
 
-    url: Optional[HttpUrl] = None
+    url: HttpUrl | None = None
 
     observers: Annotated[list[Observer], Field(min_length=1, max_length=128)]
 

shepherd_core/decoder_waveform/uart.py

@@ -23,8 +23,6 @@ https://sigrok.org/wiki/Protocol_decoder:Uart
 
 from enum import Enum
 from pathlib import Path
-from typing import Optional
-from typing import Union
 
 import numpy as np
 
@@ -51,12 +49,13 @@ class Uart:
 
     def __init__(
         self,
-        content: Union[Path, np.ndarray],
-        baud_rate: Optional[int] = None,
-        frame_length: Optional[int] = 8,
-        inversion: Optional[bool] = None,
-        parity: Optional[Parity] = Parity.no,
-        bit_order: Optional[BitOrder] = BitOrder.lsb_first,
+        content: Path | np.ndarray,
+        *,
+        baud_rate: int | None = None,
+        frame_length: int | None = 8,
+        inversion: bool | None = None,
+        parity: Parity | None = Parity.no,
+        bit_order: BitOrder | None = BitOrder.lsb_first,
     ) -> None:
         """Provide a file with two columns: TS & Signal.
 
@@ -115,9 +114,9 @@ class Uart:
             log.error("Signal still inverted?!? Check parameters and input")
 
         # results
-        self.events_symbols: Optional[np.ndarray] = None
-        self.events_lines: Optional[np.ndarray] = None
-        self.text: Optional[str] = None
+        self.events_symbols: np.ndarray | None = None
+        self.events_lines: np.ndarray | None = None
+        self.text: str | None = None
 
     def _convert_analog2digital(self, *, invert: bool = False) -> None:
         """Divide dimension in two, divided by mean-value."""
@@ -208,9 +207,9 @@ class Uart:
         if self.events_symbols is not None:
             return self.events_symbols
 
-        pos_df: Optional[int] = None
+        pos_df: int | None = None
         symbol: int = 0
-        t_start: Optional[float] = None
+        t_start: float | None = None
         content: list = []
 
         for time, value, steps in self.events_sig:

shepherd_core/fw_tools/converter.py

@@ -4,7 +4,6 @@ import base64
 import hashlib
 import shutil
 from pathlib import Path
-from typing import Union
 
 import zstandard as zstd
 from pydantic import validate_call
@@ -81,7 +80,7 @@ def base64_to_hash(content: str) -> str:
 
 
 @validate_call
-def extract_firmware(data: Union[str, Path], data_type: FirmwareDType, file_path: Path) -> Path:
+def extract_firmware(data: str | Path, data_type: FirmwareDType, file_path: Path) -> Path:
     """Make embedded firmware-data usable in filesystem.
 
     - base64-string will be transformed to file

shepherd_core/fw_tools/converter_elf.py

@@ -2,7 +2,6 @@
 
 import subprocess
 from pathlib import Path
-from typing import Optional
 
 from pydantic import validate_call
 
@@ -10,7 +9,7 @@ from pydantic import validate_call
 
 
 @validate_call
-def elf_to_hex(file_elf: Path, file_hex: Optional[Path] = None) -> Path:
+def elf_to_hex(file_elf: Path, file_hex: Path | None = None) -> Path:
     """Convert ELF to hex file using objcopy."""
     if not file_elf.is_file():
         raise ValueError("Fn needs an existing file as input")

shepherd_core/fw_tools/patcher.py

@@ -1,8 +1,9 @@
 """Read and modify symbols in ELF-files."""
 
+import shutil
 from pathlib import Path
+from tempfile import TemporaryDirectory
 from typing import Annotated
-from typing import Optional
 
 from pydantic import Field
 from pydantic import validate_call
@@ -29,27 +30,32 @@ def find_symbol(file_elf: Path, symbol: str) -> bool:
         return False
     if ELF is None:
         raise RuntimeError(elf_error_text)
-    elf = ELF(path=file_elf)
-    try:
-        addr = elf.symbols[symbol]
-    except KeyError:
-        addr = None
-    if addr is None:
-        log.debug("Symbol '%s' not found in ELF-File %s", symbol, file_elf.name)
-        return False
-    log.debug(
-        "Symbol '%s' found in ELF-File %s, arch=%s, order=%s",
-        symbol,
-        file_elf.name,
-        elf.arch,
-        elf.endian,
-    )
-    elf.close()
+    with TemporaryDirectory(ignore_cleanup_errors=True) as tmp:
+        # switcheroo that might prevent windows bug - overwrite fails in modify_symbol_value()
+        file_tmp = Path(tmp) / file_elf.name
+        shutil.copy(file_elf, file_tmp)
+        elf = ELF(path=file_tmp)
+        try:
+            addr = elf.symbols[symbol]
+        except KeyError:
+            addr = None
+        if addr is None:
+            elf.close()  # better be safe
+            log.debug("Symbol '%s' not found in ELF-File %s", symbol, file_elf.name)
+            return False
+        log.debug(
+            "Symbol '%s' found in ELF-File %s, arch=%s, order=%s",
+            symbol,
+            file_elf.name,
+            elf.arch,
+            elf.endian,
+        )
+        elf.close()
     return True
 
 
 @validate_call
-def read_symbol(file_elf: Path, symbol: str, length: int) -> Optional[int]:
+def read_symbol(file_elf: Path, symbol: str, length: int) -> int | None:
     """Read value of symbol in ELF-File.
 
     Will be interpreted as int.
@@ -61,24 +67,28 @@ def read_symbol(file_elf: Path, symbol: str, length: int) -> Optional[int]:
     elf = ELF(path=file_elf)
     addr = elf.symbols[symbol]
     value_raw = elf.read(address=addr, count=length)[-length:]
+    endian = elf.endian
     elf.close()
-    return int.from_bytes(bytes=value_raw, byteorder=elf.endian, signed=False)
+    return int.from_bytes(bytes=value_raw, byteorder=endian, signed=False)
 
 
-def read_uid(file_elf: Path) -> Optional[int]:
+def read_uid(file_elf: Path) -> int | None:
     """Read value of UID-symbol for shepherd testbed."""
     return read_symbol(file_elf, symbol=config.UID_NAME, length=config.UID_SIZE)
 
 
-def read_arch(file_elf: Path) -> Optional[str]:
+def read_arch(file_elf: Path) -> str | None:
     """Determine chip-architecture from elf-metadata."""
     if not is_elf(file_elf):
         return None
     if ELF is None:
         raise RuntimeError(elf_error_text)
     elf = ELF(path=file_elf)
-    if "exec" in elf.elftype.lower():
-        return elf.arch.lower()
+    elf_type = elf.elftype.lower()
+    elf_arch = elf.arch.lower()
+    elf.close()
+    if "exec" in elf_type:
+        return elf_arch
     log.error("ELF is not Executable")
     return None
 
@@ -90,7 +100,7 @@ def modify_symbol_value(
     value: Annotated[int, Field(ge=0, lt=2 ** (8 * config.UID_SIZE))],
     *,
     overwrite: bool = False,
-) -> Optional[Path]:
+) -> Path | None:
     """Replace value of uint16-symbol in ELF-File.
 
     Hardcoded for uint16_t (2 byte).
@@ -103,22 +113,37 @@ def modify_symbol_value(
         return None
     if ELF is None:
         raise RuntimeError(elf_error_text)
-    elf = ELF(path=file_elf)
-    addr = elf.symbols[symbol]
-    value_raw = elf.read(address=addr, count=config.UID_SIZE)[-config.UID_SIZE :]
-    # ⤷ cutting needed -> msp produces 4b instead of 2
-    value_old = int.from_bytes(bytes=value_raw, byteorder=elf.endian, signed=False)
-    value_raw = value.to_bytes(length=config.UID_SIZE, byteorder=elf.endian, signed=False)
-
-    try:
-        elf.write(address=addr, data=value_raw)
-    except AttributeError:
-        log.warning("ELF-Modifier failed @%s for symbol '%s'", f"0x{addr:X}", symbol)
-        return None
+    with TemporaryDirectory(ignore_cleanup_errors=True) as tmp:
+        # switcheroo that also prevents windows bug (overwrite fails)
+        file_tmp = Path(tmp) / file_elf.name
+        shutil.copy(file_elf, file_tmp)
+
+        elf = ELF(path=file_elf)
+        addr = elf.symbols[symbol]
+        value_raw = elf.read(address=addr, count=config.UID_SIZE)[-config.UID_SIZE :]
+        # ⤷ cutting needed -> msp produces 4b instead of 2
+        value_old = int.from_bytes(bytes=value_raw, byteorder=elf.endian, signed=False)
+        value_raw = value.to_bytes(length=config.UID_SIZE, byteorder=elf.endian, signed=False)
+
+        try:
+            elf.write(address=addr, data=value_raw)
+        except AttributeError:
+            log.warning("ELF-Modifier failed @%s for symbol '%s'", f"0x{addr:X}", symbol)
+            elf.close()
+            return None
+
+        file_new = file_elf if overwrite else file_elf.with_stem(file_elf.stem + "_" + str(value))
+        try:
+            file_new.unlink(missing_ok=True)
+        except PermissionError:
+            elf.close()
+            log.error(
+                "Failed to overwrite file, because it's somehow still in use (typical for WinOS)."
+            )
+            return None
+        elf.save(path=file_new)
+        elf.close()
 
-    file_new = file_elf if overwrite else file_elf.with_stem(file_elf.stem + "_" + str(value))
-    elf.save(path=file_new)
-    elf.close()
     log.debug(
         "Value of Symbol '%s' modified: %s -> %s @%s",
         symbol,
@@ -129,6 +154,6 @@ def modify_symbol_value(
     return file_new
 
 
-def modify_uid(file_elf: Path, value: int) -> Optional[Path]:
+def modify_uid(file_elf: Path, value: int) -> Path | None:
     """Replace value of UID-symbol for shepherd testbed."""
     return modify_symbol_value(file_elf, symbol=config.UID_NAME, value=value, overwrite=True)

shepherd_core/fw_tools/validation.py

@@ -5,6 +5,7 @@ TODO: Work in Progress.
     - detection-functions that register in main validator.
 """
 
+import shutil
 import tempfile
 from pathlib import Path
 
@@ -92,7 +93,12 @@ def is_elf(file: Path) -> bool:
     if not file.is_file():
         return False
     try:
-        _ = ELF(path=file)
+        with tempfile.TemporaryDirectory(ignore_cleanup_errors=True) as tmp:
+            # switcheroo that might prevent windows bug - overwrite fails in modify_symbol_value()
+            file_tmp = Path(tmp) / file.name
+            shutil.copy(file, file_tmp)
+            elf = ELF(path=file_tmp)
+            elf.close()
     except ELFError:
         log.debug("File %s is not ELF - Magic number does not match", file.name)
         return False