Merge pull request #542 from powerapi-ng/refactor/remove-commented-code

refactor: Remove commented-out code

Author: gfieni

.ruff.toml

@@ -19,14 +19,12 @@ ignore = [
     "B006",   # mutable-argument-default
     "B008",   # function-call-in-default-argument
     "F401",   # unused-import
-    "ERA001", # commented-out-code
     "G003",   # logging-string-concat
     "G004",   # logging-f-string
     "RUF005", # collection-literal-concatenation
     "RUF013", # implicit-optional
     "RUF015", # unnecessary-iterable-allocation-for-first-element
 
-    # "UP026",  # deprecated-mock-import
     "B005",   # strip-with-multi-characters
     "B010",   # set-attr-with-constant
     "B011",   # assert-false

src/powerapi/actor/actor.py

@@ -94,7 +94,7 @@ def __init__(self, name, level_logger=logging.WARNING, timeout=None):
                             timeout (in ms)
         """
         multiprocessing.Process.__init__(self, name=name)
-        # self.daemon = True
+
         #: (logging.Logger): Logger
         self.logger = logging.getLogger(name)
         self.logger.setLevel(level_logger)
@@ -103,14 +103,7 @@ def __init__(self, name, level_logger=logging.WARNING, timeout=None):
             '%(process)d %(processName)s || %(message)s')
         handler = logging.StreamHandler()
         handler.setFormatter(formatter)
-
-        # file logger
-        # handlerf = logging.FileHandler('powerapi.log')
-        # handlerf.setLevel(level_logger)
-        # handlerf.setFormatter(formatter)
-
         self.logger.addHandler(handler)
-        # self.logger.addHandler(handlerf)
 
         #: (powerapi.State): Actor context
         self.state = State(self)

src/powerapi/cli/config_parser.py

@@ -706,8 +706,7 @@ def _extract_group_name_from_prefix(self, prefix: str) -> str:
         are separated by self.default_separator_env_vars_names and the group name is composed by a single word
 
         """
-        # prefix.split = ['<prefix-begining>', '<group-name>', '']
-        return prefix.split(self.default_separator_env_vars_names)[1].lower()
+        return prefix.split(self.default_separator_env_vars_names)[1].lower()  # [<prefix-beginning>, <group-name>, '']
 
     def _extract_simple_environment_variables_with_prefix(self, simple_variables_prefix: str,
                                                           groups_variables_prefix: list) -> dict:

src/powerapi/database/csvdb.py

@@ -70,13 +70,6 @@ def __init__(self, db, filenames, report_type, stream_mode):
         super().__init__(db, report_type, stream_mode)
 
         self.filenames = filenames
-
-        # intern memory for reading
-        # path_file: {
-        #     'next_line': [],
-        #     'reader': None,
-        #     'file': None
-        # }
         self.tmp_read = {}
 
         # Add it in the tmp