From b8cea3f9616664fb2c61b9afc182e1f6b092d5d6 Mon Sep 17 00:00:00 2001 From: AllfatherHatt Date: Sat, 15 Mar 2025 18:32:12 +0100 Subject: [PATCH] Come on, read config you stupid memory! --- reginaldCog/memory.py | 8 ++++++-- reginaldCog/reginald.py | 9 ++------- 2 files changed, 8 insertions(+), 9 deletions(-) diff --git a/reginaldCog/memory.py b/reginaldCog/memory.py index c389860..d622f56 100644 --- a/reginaldCog/memory.py +++ b/reginaldCog/memory.py @@ -11,11 +11,15 @@ from openai import OpenAIError class MemoryMixin: """Handles all memory-related functions for Reginald.""" - def __init__(self, config: Config): - self.config = config # Now explicitly set + def __init__(self): + """No longer requires `config` as a parameter.""" self.short_term_memory_limit = 100 self.summary_retention_limit = 25 self.summary_retention_ratio = 0.8 + + def get_config(self): + """Access `_config` from the parent class (ReginaldCog).""" + return self._config # ✅ Use `_config` instead of `self.config` @commands.command(name="reginald_clear_short", help="Clears short-term memory for this channel.") @commands.has_permissions(administrator=True) diff --git a/reginaldCog/reginald.py b/reginaldCog/reginald.py index 9bfb87a..935037b 100644 --- a/reginaldCog/reginald.py +++ b/reginaldCog/reginald.py @@ -15,14 +15,9 @@ from .memory import MemoryMixin class ReginaldCog(commands.Cog, PermissionsMixin, BlacklistMixin, MemoryMixin): def __init__(self, bot): self.bot = bot - config_instance = Config.get_conf(self, identifier=71717171171717) # ✅ Create config instance + self._config = Config.get_conf(self, identifier=71717171171717) # ✅ Store `_config` first - # ✅ Pass config explicitly to MemoryMixin - MemoryMixin.__init__(self, config_instance) - - super().__init__() # ✅ Now it's safe to initialize parent classes - - self.config = config_instance # ✅ Store config after parent init + super().__init__() # Trying to get MemoryMixin to have access to _config self.default_listening_channel = 1085649787388428370 self.memory_locks = {}