Use the -O flag for improved performance in error handling #31 (#141)

* Replacing error code with optimized code

Replacing common error code with more optimize code using assert / __debug__ for production ready build along with maintaining error handling capabilities for development

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Updating few changes

fixing optimization code

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* fixed build test

fixed build test

* fixed build code

fixed build code

* updating agent to fix build

updating agent to fix build

* Updated assert code along with __debug__

Updated assert code along with __debug__ to make code informtive while in debug mode but optimized when build with -O option

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Instead of assert raising informative errors

modifying assert code with raise... to raise informative errors to the user

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update examples/sugarscape_ig/ss_polars/agents.py

Co-authored-by: Copilot <[email protected]>

* Update examples/sugarscape_ig/ss_polars/agents.py

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Adam Amer <[email protected]>
Co-authored-by: Copilot <[email protected]>

Author: 4 people

mesa_frames/abstract/agents.py

@@ -998,10 +998,11 @@ def __iadd__(self, other: DataFrame | Sequence[Any] | dict[str, Any]) -> Self:
 
     @abstractmethod
     def __getattr__(self, name: str) -> Any:
-        if name == "_agents":
-            raise RuntimeError(
-                "The _agents attribute is not set. You probably forgot to call super().__init__ in the __init__ method."
-            )
+        if __debug__:  # Only execute in non-optimized mode
+            if name == "_agents":
+                raise AttributeError(
+                    "The _agents attribute is not set. You probably forgot to call super().__init__ in the __init__ method."
+                )
 
     @overload
     def __getitem__(self, key: str | tuple[AgentMask, str]) -> Series | DataFrame: ...

mesa_frames/concrete/agents.py

@@ -438,10 +438,12 @@ def __add__(self, other: AgentSetDF | Iterable[AgentSetDF]) -> Self:
         return super().__add__(other)
 
     def __getattr__(self, name: str) -> dict[AgentSetDF, Any]:
-        if name.startswith("_"):  # Avoids infinite recursion of private attributes
-            raise AttributeError(
-                f"'{self.__class__.__name__}' object has no attribute '{name}'"
-            )
+        # Avoids infinite recursion of private attributes
+        if __debug__:  # Only execute in non-optimized mode
+            if name.startswith("_"):
+                raise AttributeError(
+                    f"'{self.__class__.__name__}' object has no attribute '{name}'"
+                )
         return {agentset: getattr(agentset, name) for agentset in self._agentsets}
 
     @overload

mesa_frames/concrete/model.py

@@ -175,14 +175,17 @@ def agents(self) -> AgentsDF:
         try:
             return self._agents
         except AttributeError:
-            raise ValueError(
-                "You haven't called super().__init__() in your model. Make sure to call it in your __init__ method."
-            )
+            if __debug__:  # Only execute in non-optimized mode
+                raise RuntimeError(
+                    "You haven't called super().__init__() in your model. Make sure to call it in your __init__ method."
+                )
 
     @agents.setter
     def agents(self, agents: AgentsDF) -> None:
-        if not isinstance(agents, AgentsDF):
-            raise TypeError("agents must be an instance of AgentsDF")
+        if __debug__:  # Only execute in non-optimized mode
+            if not isinstance(agents, AgentsDF):
+                raise TypeError("agents must be an instance of AgentsDF")
+
         self._agents = agents
 
     @property