Skip to content

Commit f640855

Browse files
authored
Fix: fix pg query for group error string (#465)
* fix:get_grouped_counts * fix: code format
1 parent 5e4f695 commit f640855

File tree

3 files changed

+14
-11
lines changed

3 files changed

+14
-11
lines changed

src/memos/api/config.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -861,9 +861,9 @@ def create_user_config(user_name: str, user_id: str) -> tuple[MOSConfig, General
861861
"reorganize": os.getenv("MOS_ENABLE_REORGANIZE", "false").lower()
862862
== "true",
863863
"memory_size": {
864-
"WorkingMemory": os.getenv("NEBULAR_WORKING_MEMORY", 20),
865-
"LongTermMemory": os.getenv("NEBULAR_LONGTERM_MEMORY", 1e6),
866-
"UserMemory": os.getenv("NEBULAR_USER_MEMORY", 1e6),
864+
"WorkingMemory": int(os.getenv("NEBULAR_WORKING_MEMORY", 20)),
865+
"LongTermMemory": int(os.getenv("NEBULAR_LONGTERM_MEMORY", 1e6)),
866+
"UserMemory": int(os.getenv("NEBULAR_USER_MEMORY", 1e6)),
867867
},
868868
"search_strategy": {
869869
"fast_graph": bool(os.getenv("FAST_GRAPH", "false") == "true"),
@@ -933,9 +933,9 @@ def get_default_cube_config() -> GeneralMemCubeConfig | None:
933933
== "true",
934934
"internet_retriever": internet_config,
935935
"memory_size": {
936-
"WorkingMemory": os.getenv("NEBULAR_WORKING_MEMORY", 20),
937-
"LongTermMemory": os.getenv("NEBULAR_LONGTERM_MEMORY", 1e6),
938-
"UserMemory": os.getenv("NEBULAR_USER_MEMORY", 1e6),
936+
"WorkingMemory": int(os.getenv("NEBULAR_WORKING_MEMORY", 20)),
937+
"LongTermMemory": int(os.getenv("NEBULAR_LONGTERM_MEMORY", 1e6)),
938+
"UserMemory": int(os.getenv("NEBULAR_USER_MEMORY", 1e6)),
939939
},
940940
"search_strategy": {
941941
"fast_graph": bool(os.getenv("FAST_GRAPH", "false") == "true"),

src/memos/graph_dbs/polardb.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1740,9 +1740,11 @@ def get_grouped_counts(
17401740
for field in group_fields:
17411741
alias = field.replace(".", "_")
17421742
return_fields.append(
1743-
f"ag_catalog.agtype_access_operator(properties, '\"{field}\"'::agtype) AS {alias}"
1743+
f"ag_catalog.agtype_access_operator(properties, '\"{field}\"'::agtype)::text AS {alias}"
1744+
)
1745+
group_by_fields.append(
1746+
f"ag_catalog.agtype_access_operator(properties, '\"{field}\"'::agtype)::text"
17441747
)
1745-
group_by_fields.append(alias)
17461748

17471749
# Full SQL query construction
17481750
query = f"""
@@ -1751,7 +1753,6 @@ def get_grouped_counts(
17511753
{where_clause}
17521754
GROUP BY {", ".join(group_by_fields)}
17531755
"""
1754-
17551756
conn = self._get_connection()
17561757
try:
17571758
with conn.cursor() as cursor:
@@ -1772,7 +1773,7 @@ def get_grouped_counts(
17721773
else:
17731774
group_values[field] = str(value)
17741775
count_value = row[-1] # Last column is count
1775-
output.append({**group_values, "count": count_value})
1776+
output.append({**group_values, "count": int(count_value)})
17761777

17771778
return output
17781779

src/memos/memories/textual/tree_text_memory/organize/manager.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,9 @@ def _refresh_memory_size(self, user_name: str | None = None) -> None:
156156
results = self.graph_store.get_grouped_counts(
157157
group_fields=["memory_type"], user_name=user_name
158158
)
159-
self.current_memory_size = {record["memory_type"]: record["count"] for record in results}
159+
self.current_memory_size = {
160+
record["memory_type"]: int(record["count"]) for record in results
161+
}
160162
logger.info(f"[MemoryManager] Refreshed memory sizes: {self.current_memory_size}")
161163

162164
def _process_memory(self, memory: TextualMemoryItem, user_name: str | None = None):

0 commit comments

Comments
 (0)