From dbf41120b0999aafdfec32cd4617bbdd3290874a Mon Sep 17 00:00:00 2001
From: seehi <6580@pm.me>
Date: Wed, 28 Aug 2024 11:46:05 +0800
Subject: [PATCH] update setter of is_writable and is_readable

---
 metagpt/exp_pool/manager.py | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/metagpt/exp_pool/manager.py b/metagpt/exp_pool/manager.py
index 38772239b..35de17079 100644
--- a/metagpt/exp_pool/manager.py
+++ b/metagpt/exp_pool/manager.py
@@ -49,7 +49,11 @@ class ExperienceManager(BaseModel):
 
     @is_readable.setter
     def is_readable(self, value: bool):
-        self.config.exp_pool.enabled = self.config.exp_pool.enable_read = value
+        self.config.exp_pool.enable_read = value
+
+        # If set to True, ensure that enabled is also True.
+        if value:
+            self.config.exp_pool.enabled = True
 
     @property
     def is_writable(self) -> bool:
@@ -57,7 +61,11 @@ class ExperienceManager(BaseModel):
 
     @is_writable.setter
     def is_writable(self, value: bool):
-        self.config.exp_pool.enabled = self.config.exp_pool.enable_write = value
+        self.config.exp_pool.enable_write = value
+
+        # If set to True, ensure that enabled is also True.
+        if value:
+            self.config.exp_pool.enabled = True
 
     @handle_exception
     def create_exp(self, exp: Experience):