feat: implement self-reflection loop in graph builder and unify models to gpt-4o-mini
Browse files
src/graphBuilder/neo4j/finGraph.py
CHANGED
|
@@ -50,7 +50,7 @@ def get_neo4j_driver() -> neo4j.Driver:
|
|
| 50 |
driver = None
|
| 51 |
|
| 52 |
chat_llm = ChatOpenAI(model="gpt-4o-mini", temperature=0)
|
| 53 |
-
rag_llm = OpenAILLM(model_name="gpt-4o", model_params={"temperature": 0})
|
| 54 |
embedder = OpenAIEmbeddings(model="text-embedding-3-small")
|
| 55 |
|
| 56 |
INDEX_NAME = "content_vector_index"
|
|
@@ -67,6 +67,8 @@ class ArticleState(TypedDict):
|
|
| 67 |
is_ai_related: bool
|
| 68 |
entities: List[Dict]
|
| 69 |
relations: List[Dict]
|
|
|
|
|
|
|
| 70 |
|
| 71 |
|
| 72 |
def check_ai_relevance(state: ArticleState) -> ArticleState:
|
|
@@ -83,8 +85,19 @@ def check_ai_relevance(state: ArticleState) -> ArticleState:
|
|
| 83 |
|
| 84 |
|
| 85 |
def extract_entities(state: ArticleState) -> ArticleState:
|
| 86 |
-
"""Node 2: μν°ν° μΆμΆ"""
|
| 87 |
-
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 88 |
μν°ν° μ ν:
|
| 89 |
- AICompany: κΈ°μ
/κΈ°κ΄ (μ: μΌμ±μ μ, OpenAI)
|
| 90 |
- AITechnology: AI κΈ°μ (μ: λκ·λͺ¨μΈμ΄λͺ¨λΈ, κ°ννμ΅)
|
|
@@ -92,18 +105,58 @@ def extract_entities(state: ArticleState) -> ArticleState:
|
|
| 92 |
- AIField: μ μ© λΆμΌ (μ: κΈμ΅AI, AI λ°λ체)
|
| 93 |
|
| 94 |
μ λͺ©: {state["title"]}
|
| 95 |
-
λ³Έλ¬Έ: {state["text"][:900]}
|
| 96 |
|
| 97 |
-
JSONμΌλ‘λ§ μλ΅:{{"entities":[{{"name":"...","type":"AICompany|AITechnology|AIService|AIField","description":"..."}}]}}"""
|
|
|
|
| 98 |
res = chat_llm.invoke(prompt)
|
|
|
|
|
|
|
|
|
|
| 99 |
try:
|
| 100 |
raw = str(res.content).strip()
|
| 101 |
if "```" in raw:
|
| 102 |
raw = raw.split("```")[1].lstrip("json")
|
| 103 |
-
|
| 104 |
-
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 105 |
entities = []
|
| 106 |
-
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 107 |
|
| 108 |
|
| 109 |
def extract_relations(state: ArticleState) -> ArticleState:
|
|
@@ -134,13 +187,40 @@ def route_after_check(state: ArticleState) -> str:
|
|
| 134 |
return "extract_entities" if state["is_ai_related"] else END
|
| 135 |
|
| 136 |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 137 |
builder = StateGraph(ArticleState)
|
| 138 |
builder.add_node("check_ai", check_ai_relevance)
|
| 139 |
builder.add_node("extract_entities", extract_entities)
|
| 140 |
builder.add_node("extract_relations", extract_relations)
|
| 141 |
builder.set_entry_point("check_ai")
|
| 142 |
builder.add_conditional_edges("check_ai", route_after_check)
|
| 143 |
-
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| 144 |
builder.add_edge("extract_relations", END)
|
| 145 |
pipeline = builder.compile()
|
| 146 |
|
|
@@ -303,6 +383,8 @@ def main() -> None:
|
|
| 303 |
is_ai_related=False,
|
| 304 |
entities=[],
|
| 305 |
relations=[],
|
|
|
|
|
|
|
| 306 |
)
|
| 307 |
out = pipeline.invoke(state)
|
| 308 |
if out["is_ai_related"]:
|
|
|
|
| 50 |
driver = None
|
| 51 |
|
| 52 |
chat_llm = ChatOpenAI(model="gpt-4o-mini", temperature=0)
|
| 53 |
+
rag_llm = OpenAILLM(model_name="gpt-4o-mini", model_params={"temperature": 0})
|
| 54 |
embedder = OpenAIEmbeddings(model="text-embedding-3-small")
|
| 55 |
|
| 56 |
INDEX_NAME = "content_vector_index"
|
|
|
|
| 67 |
is_ai_related: bool
|
| 68 |
entities: List[Dict]
|
| 69 |
relations: List[Dict]
|
| 70 |
+
retry_count: int
|
| 71 |
+
reflection_feedback: str
|
| 72 |
|
| 73 |
|
| 74 |
def check_ai_relevance(state: ArticleState) -> ArticleState:
|
|
|
|
| 85 |
|
| 86 |
|
| 87 |
def extract_entities(state: ArticleState) -> ArticleState:
|
| 88 |
+
"""Node 2: μν°ν° μΆμΆ (μκΈ°λ°μ± νΌλλ°± λ°μ λ° νμ
μ ν©μ± κ²μ¦)"""
|
| 89 |
+
retry_count = state.get("retry_count", 0) + 1
|
| 90 |
+
feedback = state.get("reflection_feedback", "")
|
| 91 |
+
|
| 92 |
+
feedback_prompt = ""
|
| 93 |
+
if feedback:
|
| 94 |
+
feedback_prompt = (
|
| 95 |
+
f"\n\nβ οΈ [μ΄μ μλμ λν κ²μ¦ μ€λ₯ νΌλλ°±]:\n{feedback}\n"
|
| 96 |
+
"μ μ€λ₯λ₯Ό λ°λμ λΆμνμ¬, μ΄λ²μλ μ€λ³΅λκ±°λ λΉμ΄μκ±°λ λΆμμ νμ§ μκ³ "
|
| 97 |
+
"μ νν νμ
κ³Ό μ€λͺ
μ κ°μΆ μ¬λ°λ₯Έ μν°ν°λ§ μ격νκ² JSONμΌλ‘ μΆμΆν΄μ£ΌμΈμ."
|
| 98 |
+
)
|
| 99 |
+
|
| 100 |
+
prompt = f"""λ€μ AI λ΄μ€μμ ν΅μ¬ μν°ν°λ€μ μΆμΆνμΈμ.
|
| 101 |
μν°ν° μ ν:
|
| 102 |
- AICompany: κΈ°μ
/κΈ°κ΄ (μ: μΌμ±μ μ, OpenAI)
|
| 103 |
- AITechnology: AI κΈ°μ (μ: λκ·λͺ¨μΈμ΄λͺ¨λΈ, κ°ννμ΅)
|
|
|
|
| 105 |
- AIField: μ μ© λΆμΌ (μ: κΈμ΅AI, AI λ°λ체)
|
| 106 |
|
| 107 |
μ λͺ©: {state["title"]}
|
| 108 |
+
λ³Έλ¬Έ: {state["text"][:900]}{feedback_prompt}
|
| 109 |
|
| 110 |
+
JSONμΌλ‘λ§ μλ΅: {{"entities":[{{"name":"...","type":"AICompany|AITechnology|AIService|AIField","description":"..."}}]}}"""
|
| 111 |
+
|
| 112 |
res = chat_llm.invoke(prompt)
|
| 113 |
+
entities = []
|
| 114 |
+
new_feedback = ""
|
| 115 |
+
|
| 116 |
try:
|
| 117 |
raw = str(res.content).strip()
|
| 118 |
if "```" in raw:
|
| 119 |
raw = raw.split("```")[1].lstrip("json")
|
| 120 |
+
data = json.loads(raw)
|
| 121 |
+
extracted = data.get("entities", [])
|
| 122 |
+
|
| 123 |
+
allowed_types = {"AICompany", "AITechnology", "AIService", "AIField"}
|
| 124 |
+
valid_entities = []
|
| 125 |
+
for e in extracted:
|
| 126 |
+
name = e.get("name", "").strip()
|
| 127 |
+
etype = e.get("type", "").strip()
|
| 128 |
+
desc = e.get("description", "").strip()
|
| 129 |
+
|
| 130 |
+
if not name:
|
| 131 |
+
new_feedback += "- μν°ν°μ μ΄λ¦(name) νλκ° λλ½λμκ±°λ λΉμ΄μμ΅λλ€.\n"
|
| 132 |
+
continue
|
| 133 |
+
if etype not in allowed_types:
|
| 134 |
+
new_feedback += f"- μν°ν° '{name}'μ νμ
'{etype}'μ νμ©λ μ’
λ₯({', '.join(allowed_types)})κ° μλλλ€.\n"
|
| 135 |
+
continue
|
| 136 |
+
if not desc:
|
| 137 |
+
new_feedback += f"- μν°ν° '{name}'μ λν μ€λͺ
(description)μ΄ μλ΅λμμ΅λλ€.\n"
|
| 138 |
+
continue
|
| 139 |
+
|
| 140 |
+
valid_entities.append({
|
| 141 |
+
"name": name,
|
| 142 |
+
"type": etype,
|
| 143 |
+
"description": desc
|
| 144 |
+
})
|
| 145 |
+
|
| 146 |
+
entities = valid_entities
|
| 147 |
+
if not entities:
|
| 148 |
+
new_feedback = "μ ν¨ν μν°ν°κ° νλλ μΆμΆλμ§ μμμ΅λλ€."
|
| 149 |
+
|
| 150 |
+
except Exception as err:
|
| 151 |
entities = []
|
| 152 |
+
new_feedback = f"μλ΅ JSON νμ± μ€ν¨ λλ νμμ΄ μ¬λ°λ₯΄μ§ μμ΅λλ€. μλ¬: {str(err)}"
|
| 153 |
+
|
| 154 |
+
return {
|
| 155 |
+
**state,
|
| 156 |
+
"entities": entities,
|
| 157 |
+
"retry_count": retry_count,
|
| 158 |
+
"reflection_feedback": new_feedback.strip()
|
| 159 |
+
}
|
| 160 |
|
| 161 |
|
| 162 |
def extract_relations(state: ArticleState) -> ArticleState:
|
|
|
|
| 187 |
return "extract_entities" if state["is_ai_related"] else END
|
| 188 |
|
| 189 |
|
| 190 |
+
def validate_entities(state: ArticleState) -> str:
|
| 191 |
+
"""μΆμΆλ μν°ν°μ νμ§μ κ²μ¦νκ³ , λ―Έλ¬ν κ²½μ° μ΅λ 3νκΉμ§ μκΈ°λ°μ±(Self-Reflection) 루νλ₯Ό λμμν΅λλ€."""
|
| 192 |
+
retry_count = state.get("retry_count", 0)
|
| 193 |
+
feedback = state.get("reflection_feedback", "")
|
| 194 |
+
entities = state.get("entities", [])
|
| 195 |
+
|
| 196 |
+
# μΆμΆμ λ¬Έμ μ μ΄ μκ³ μμ§ μ΅λ 3ν μ¬μλλ₯Ό μ΄κ³Όνμ§ μμ κ²½μ°
|
| 197 |
+
if (feedback or not entities) and retry_count < 3:
|
| 198 |
+
print(f" β οΈ [Self-Reflection] μν°ν° νμ§ λ―Έλ¬ (μλ {retry_count}/3). νΌλλ°±: {feedback[:100]}...")
|
| 199 |
+
return "extract_entities" # μκΈ°λ°μ± 루νλ‘ λ³΅κ·
|
| 200 |
+
|
| 201 |
+
if feedback and retry_count >= 3:
|
| 202 |
+
print(f" π¨ [Self-Reflection] μν°ν° 3ν μλ μ΄κ³Ό. κ²μ¦ μ€λ₯κ° μμ§λ§ ν¨μ€ν©λλ€. νΌλλ°±: {feedback[:100]}...")
|
| 203 |
+
|
| 204 |
+
return "extract_relations" # κ²μ¦μ μ μ ν΅κ³Όνκ±°λ μ΅λ 3ν νλμ λλ¬ν κ²½μ° ν΅κ³Ό
|
| 205 |
+
|
| 206 |
+
|
| 207 |
builder = StateGraph(ArticleState)
|
| 208 |
builder.add_node("check_ai", check_ai_relevance)
|
| 209 |
builder.add_node("extract_entities", extract_entities)
|
| 210 |
builder.add_node("extract_relations", extract_relations)
|
| 211 |
builder.set_entry_point("check_ai")
|
| 212 |
builder.add_conditional_edges("check_ai", route_after_check)
|
| 213 |
+
|
| 214 |
+
# μκΈ°λ°μ± μ‘°κ±΄λΆ μ£μ§ λ§€ν
|
| 215 |
+
builder.add_conditional_edges(
|
| 216 |
+
"extract_entities",
|
| 217 |
+
validate_entities,
|
| 218 |
+
{
|
| 219 |
+
"extract_entities": "extract_entities",
|
| 220 |
+
"extract_relations": "extract_relations"
|
| 221 |
+
}
|
| 222 |
+
)
|
| 223 |
+
|
| 224 |
builder.add_edge("extract_relations", END)
|
| 225 |
pipeline = builder.compile()
|
| 226 |
|
|
|
|
| 383 |
is_ai_related=False,
|
| 384 |
entities=[],
|
| 385 |
relations=[],
|
| 386 |
+
retry_count=0,
|
| 387 |
+
reflection_feedback="",
|
| 388 |
)
|
| 389 |
out = pipeline.invoke(state)
|
| 390 |
if out["is_ai_related"]:
|
src/retrieval/finRetrieval.py
CHANGED
|
@@ -38,7 +38,7 @@ class HybridResult:
|
|
| 38 |
"""GraphRAG λλ μΌλ° μ§μ κΈ°λ° ν΅ν© μλ΅ κ²°κ³Ό"""
|
| 39 |
|
| 40 |
answer: str # μ΅μ’
λ΅λ³ λ¬Έμμ΄
|
| 41 |
-
mode: str # "graph": κ·Έλν κ²μ κΈ°λ° | "general": GPT-4o μΌλ° μ§μ κΈ°λ°
|
| 42 |
retriever_result: Any = None # RetrieverResult (mode="graph"μΌ λλ§ μ ν¨)
|
| 43 |
|
| 44 |
|
|
@@ -294,7 +294,7 @@ class LazyGraphRAG:
|
|
| 294 |
return
|
| 295 |
|
| 296 |
# OpenAI ν΄λΌμ΄μΈνΈ λ° μλ² λ μ§μ° μ΄κΈ°ν (CI ν¬λμ λ°©μ§)
|
| 297 |
-
self._rag_llm = OpenAILLM(model_name="gpt-4o", model_params={"temperature": 0})
|
| 298 |
embedder = OpenAIEmbeddings(model="text-embedding-3-small")
|
| 299 |
|
| 300 |
driver = get_neo4j_driver()
|
|
@@ -349,7 +349,7 @@ class LazyGraphRAG:
|
|
| 349 |
)
|
| 350 |
|
| 351 |
def _is_context_sufficient(self, query_text: str, history: list, retriever_result: Any) -> bool:
|
| 352 |
-
"""κ²μλ 컨ν
μ€νΈκ° μ§λ¬Έ λ° μ΄μ λν νλ¦μ μ€μ§μ μΌλ‘ λμμ΄ λλ κΈμ΅/κΈ°μ λ΄μ€ λ°μ΄ν°μΈμ§ GPT-4oλ‘ νλ¨"""
|
| 353 |
if retriever_result is None:
|
| 354 |
return False
|
| 355 |
if not hasattr(retriever_result, "items") or not retriever_result.items:
|
|
@@ -360,7 +360,7 @@ class LazyGraphRAG:
|
|
| 360 |
if len(total_content) < 100:
|
| 361 |
return False
|
| 362 |
|
| 363 |
-
# GPT-4o κΈ°λ° μ§λ₯μ μκ° μ§λ¨ (μ΄μ λν νμ€ν 리 λ° μ§λ¬Έμ λ§₯λ½ κ²°ν© νμ )
|
| 364 |
try:
|
| 365 |
assert self._rag_llm is not None
|
| 366 |
context_snippet = total_content[:800]
|
|
@@ -414,12 +414,12 @@ class LazyGraphRAG:
|
|
| 414 |
return normalized
|
| 415 |
|
| 416 |
def _generate_general_answer(self, query_text: str, history: list) -> str:
|
| 417 |
-
"""κ·Έλν κ²μ κ²°κ³Ό μμ΄ GPT-4o μΌλ° μ§μμΌλ‘ λ΅λ³ μμ± (λν νμ€ν 리 λ°μ)"""
|
| 418 |
assert self._rag_llm is not None
|
| 419 |
system_prompt = (
|
| 420 |
"λΉμ μ AI λ° νν
ν¬ κΈ°μ νΈλ λ μ λ¬Έκ°μ΄μ, μ·¨μ
μ€λΉμμ μλ λΆμμ λλ μ λ΅ μ»¨μ€ν΄νΈμ
λλ€.\n"
|
| 421 |
"νμ¬ FinGraph μ§μ κ·Έλν(Neo4j GraphRAG)μμ κ΄λ ¨ λ΄μ€ κΈ°μ¬λ₯Ό μ°Ύμ§ λͺ»νμ΅λλ€.\n"
|
| 422 |
-
"μ΄μ λν λ§₯λ½μ μΆ©λΆν λ°μνκ³ , GPT-4oμ μΌλ° νμ΅ λ°μ΄ν°μ κΈ°λ°νμ¬ μ΅μ μ λ€ν΄ μ λ¬Έμ μΌλ‘ λ΅λ³ν΄ μ£ΌμΈμ.\n\n"
|
| 423 |
"[μ€μ μ§μΉ¨]\n"
|
| 424 |
"- μ€μ μ‘΄μ¬νμ§ μλ λ΄μ€ λ§ν¬, λ μ§, κ°μ§ URLμ μ λ μμ±νμ§ λ§μΈμ.\n"
|
| 425 |
"- κ°λ₯νλ€λ©΄ μ·¨μ
μ€λΉμμ΄ λ©΄μ /μμμμ νμ©ν μ μλ μ€μ§μ μΈ μΈμ¬μ΄νΈλ₯Ό ν¬ν¨ν΄ μ£ΌμΈμ.\n"
|
|
@@ -460,7 +460,7 @@ class LazyGraphRAG:
|
|
| 460 |
retriever_result=rag_result.retriever_result,
|
| 461 |
)
|
| 462 |
else:
|
| 463 |
-
# 3b. μΌλ° μ§μ κΈ°λ° -> νμ€ν 리 ν¬ν¨ GPT-4o μ§μ νΈμΆ
|
| 464 |
answer = self._generate_general_answer(query_text, history)
|
| 465 |
return HybridResult(answer=answer, mode="general", retriever_result=None)
|
| 466 |
|
|
|
|
| 38 |
"""GraphRAG λλ μΌλ° μ§μ κΈ°λ° ν΅ν© μλ΅ κ²°κ³Ό"""
|
| 39 |
|
| 40 |
answer: str # μ΅μ’
λ΅λ³ λ¬Έμμ΄
|
| 41 |
+
mode: str # "graph": κ·Έλν κ²μ κΈ°λ° | "general": GPT-4o-mini μΌλ° μ§μ κΈ°λ°
|
| 42 |
retriever_result: Any = None # RetrieverResult (mode="graph"μΌ λλ§ μ ν¨)
|
| 43 |
|
| 44 |
|
|
|
|
| 294 |
return
|
| 295 |
|
| 296 |
# OpenAI ν΄λΌμ΄μΈνΈ λ° μλ² λ μ§μ° μ΄κΈ°ν (CI ν¬λμ λ°©μ§)
|
| 297 |
+
self._rag_llm = OpenAILLM(model_name="gpt-4o-mini", model_params={"temperature": 0})
|
| 298 |
embedder = OpenAIEmbeddings(model="text-embedding-3-small")
|
| 299 |
|
| 300 |
driver = get_neo4j_driver()
|
|
|
|
| 349 |
)
|
| 350 |
|
| 351 |
def _is_context_sufficient(self, query_text: str, history: list, retriever_result: Any) -> bool:
|
| 352 |
+
"""κ²μλ 컨ν
μ€νΈκ° μ§λ¬Έ λ° μ΄μ λν νλ¦μ μ€μ§μ μΌλ‘ λμμ΄ λλ κΈμ΅/κΈ°μ λ΄μ€ λ°μ΄ν°μΈμ§ GPT-4o-miniλ‘ νλ¨"""
|
| 353 |
if retriever_result is None:
|
| 354 |
return False
|
| 355 |
if not hasattr(retriever_result, "items") or not retriever_result.items:
|
|
|
|
| 360 |
if len(total_content) < 100:
|
| 361 |
return False
|
| 362 |
|
| 363 |
+
# GPT-4o-mini κΈ°λ° μ§λ₯μ μκ° μ§λ¨ (μ΄μ λν νμ€ν 리 λ° μ§λ¬Έμ λ§₯λ½ κ²°ν© νμ )
|
| 364 |
try:
|
| 365 |
assert self._rag_llm is not None
|
| 366 |
context_snippet = total_content[:800]
|
|
|
|
| 414 |
return normalized
|
| 415 |
|
| 416 |
def _generate_general_answer(self, query_text: str, history: list) -> str:
|
| 417 |
+
"""κ·Έλν κ²μ κ²°κ³Ό μμ΄ GPT-4o-mini μΌλ° μ§μμΌλ‘ λ΅λ³ μμ± (λν νμ€ν 리 λ°μ)"""
|
| 418 |
assert self._rag_llm is not None
|
| 419 |
system_prompt = (
|
| 420 |
"λΉμ μ AI λ° νν
ν¬ κΈ°μ νΈλ λ μ λ¬Έκ°μ΄μ, μ·¨μ
μ€λΉμμ μλ λΆμμ λλ μ λ΅ μ»¨μ€ν΄νΈμ
λλ€.\n"
|
| 421 |
"νμ¬ FinGraph μ§μ κ·Έλν(Neo4j GraphRAG)μμ κ΄λ ¨ λ΄μ€ κΈ°μ¬λ₯Ό μ°Ύμ§ λͺ»νμ΅λλ€.\n"
|
| 422 |
+
"μ΄μ λν λ§₯λ½μ μΆ©λΆν λ°μνκ³ , GPT-4o-miniμ μΌλ° νμ΅ λ°μ΄ν°μ κΈ°λ°νμ¬ μ΅μ μ λ€ν΄ μ λ¬Έμ μΌλ‘ λ΅λ³ν΄ μ£ΌμΈμ.\n\n"
|
| 423 |
"[μ€μ μ§μΉ¨]\n"
|
| 424 |
"- μ€μ μ‘΄μ¬νμ§ μλ λ΄μ€ λ§ν¬, λ μ§, κ°μ§ URLμ μ λ μμ±νμ§ λ§μΈμ.\n"
|
| 425 |
"- κ°λ₯νλ€λ©΄ μ·¨μ
μ€λΉμμ΄ λ©΄μ /μμμμ νμ©ν μ μλ μ€μ§μ μΈ μΈμ¬μ΄νΈλ₯Ό ν¬ν¨ν΄ μ£ΌμΈμ.\n"
|
|
|
|
| 460 |
retriever_result=rag_result.retriever_result,
|
| 461 |
)
|
| 462 |
else:
|
| 463 |
+
# 3b. μΌλ° μ§μ κΈ°λ° -> νμ€ν 리 ν¬ν¨ GPT-4o-mini μ§μ νΈμΆ
|
| 464 |
answer = self._generate_general_answer(query_text, history)
|
| 465 |
return HybridResult(answer=answer, mode="general", retriever_result=None)
|
| 466 |
|