Skip to content

Commit

Permalink
Merge pull request #15 from mvinyard/bug-fix
Browse files Browse the repository at this point in the history
Fix error-prone logging
  • Loading branch information
mvinyard authored Mar 6, 2025
2 parents 6fdb151 + c726edc commit 4aa8e4c
Showing 1 changed file with 4 additions and 11 deletions.
15 changes: 4 additions & 11 deletions ABCParse/_abc_parse.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,3 @@
__module_name__ = "_abc_parse.py"
__doc__ = """Better abstract base class for auto-parsing inputs."""
__author__ = ", ".join(["Michael E. Vinyard"])
__email__ = ", ".join(["[email protected]"])


# -- import packages: ---------------------------------------------------------
import abc

Expand All @@ -15,11 +9,14 @@

# -- Controller class: --------------------------------------------------------
class ABCParse(abc.ABC):
"""
Better abstract base class for auto-parsing inputs.
"""
_BUILT = False

def __init__(self, *args, **kwargs) -> None:
"""
we avoid defining things in __init__ because this subsequently
We avoid defining things in __init__ because this subsequently
mandates the use of `super().__init__()`
Example
Expand Down Expand Up @@ -147,10 +144,6 @@ def __parse__(
for key, val in kwargs.items():
if not key in self._IGNORE:
self.__set__(key, val, public, private)

# Then log after attributes are set
self._cls_logger.debug(f"Parsing kwargs: {kwargs}")
self._cls_logger.debug(f"Parsed {len(self._PARAMS)} parameters")

def __update__(
self,
Expand Down

0 comments on commit 4aa8e4c

Please sign in to comment.