Skip to content

Commit

Permalink
add some comments
Browse files Browse the repository at this point in the history
  • Loading branch information
mschwoer committed Nov 14, 2024
1 parent d357780 commit e29d44f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions alphadia/workflow/manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@ def __init__(
self.reporter = reporting.LogBackend() if reporter is None else reporter

if load_from_file:
# Note: be careful not to overwrite loaded values by initializing them in child classes after calling super().__init__()
self.load()

@property
Expand Down
6 changes: 3 additions & 3 deletions alphadia/workflow/managers/raw_file_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ def __init__(
load_from_file: bool = False,
**kwargs,
):
"""Contains and updates timing information for the portions of the workflow."""
self.stats = {} # needs to be before super().__init__ to avoid overwriting when loading
"""Handles raw file loading and contains information on the raw file."""
self.stats = {} # needs to be before super().__init__ to avoid overwriting loaded values

super().__init__(path=path, load_from_file=load_from_file, **kwargs)
self.reporter.log_string(f"Initializing {self.__class__.__name__}")
Expand Down Expand Up @@ -50,7 +50,7 @@ def get_dia_data_object(

is_wsl = self._config["general"]["wsl"]
if is_wsl:
# copy file to /tmp # TODO why is that?
# copy file to /tmp # TODO check if WSL support can be dropped
import shutil

tmp_path = "/tmp"
Expand Down

0 comments on commit e29d44f

Please sign in to comment.