생각나는대로 냅다 다 적겠습니다ㅎㅎ가벼운 마음으로 고려해주시면 감사하겠습니다!

1. Overview

1. 설명해야할 내용이 너무 많은 주제라 고민이 많으셨을것 같아요. 핵심만 잘 정리중이신 느낌이라 좋지만 Knowledge Graph와 같이 중요하지만 한마디로 설명하기엔 너무 복잡한 내용은 잘 설명된 곳으로 링크 보내면 좀 더 친절한 글이 될 것 같습니다
2. GraphRAG의 장점을 잘 설명해주셨는데, 구축 난이도와 같은 단점은? 이라는 생각이 일부러 안넣으신건가? 라는 생각과 함께 들었습니다.

2. Environment Setup:

사소하지만, !pip install 임시방편 부분들 공통부분에 넣으면 되는걸로 알고있습니다ㅎ

# Install required packages
from langchain_opentutorial import package

package.install(
    [
        "langchain-neo4j",
        "pyalex"
    ],
    verbose=False,
    upgrade=False,
)

3. 데이터

1. 노드에 대한 설명은 잘 해주셨으나, 엣지는 어떻게 구성되어있는지 한번에 볼 수 있는 그림이 있으면 좋을것 같다는 생각이 들었습니다.
2. json 파일 내용도 어떻게 구성되어있는지 살짝 보여주시면 좋을것 같습니다.

4. 실행 코드

마지막에서 두번째 셀

chain = GraphCypherQAChain.from_llm(
    llm, graph=graph, verbose=True, qa_prompt=CYPHER_QA_PROMPT, cypher_prompt=CYPHER_GENERATION_PROMPT,
    allow_dangerous_requests = True
)

마지막에서 두 번째 셀의 from_llm 부분은 현재 방식이 오래된 것 같습니다.

예전(3~4개월 전) 기준으로는 LangChain이 해당 클래스를 업데이트하지 않았던 것으로 기억하는데, 최신 상태에서도 그대로인지 살짝 확인해보시면 좋을 것 같습니다.

완전 초안이고, 생각나는 대로 마구잡이로 적어본 내용이라 가볍게 참고만 해주시면 좋겠습니다! 고생하셨습니다!

상세한 리뷰 정말 감사드립니다🥰 다음번 커밋에 피드백주신 바를 적극 반영해 보도록 하겠습니다! 정말 감사합니다 (_ _)

저도 코드 돌려보고 정리해서 적어봅니다!

1. Overview

• 저랑 비슷한 주제인데, 구현 방식이 아예 달라서 좋은 것 같습니다! 다행히 제가 이걸 먼저 못보고(?) 만들었는데 neo4j랑 docker를 사용해서 GraphDB를 구축하고 만드는 점이 좋았습니다. 그리고 질의도 아예 GraphDB에서 쓰는 Cypher를 사용한 점도 좋았습니다.
• 다만, Academic Search System 답게 논문을 추천하는 방향으로 예시가 발전되면 좋지 않을까 합니다. (가능한지는 잘 몰라서.. 그냥 의견으로 받아들여주시면 좋을 것 같아요)

2. 데이터

• 저도 부족한 부분이지만, 데이터에 대한 설명이 더 있으면 좋을 것 같습니다. 받아놓은 json이 어떤 정보를 담고 있는지에 대한 metadata는 알지만 구체적으로 어떤 데이터인지 몰라서 사용에 고민이 될 것 같아요.
• 예를 들면, 저는 arXiv에 있는 논문을 쓰는데 그런 json 파일인지 아니면 미리 정제해놓은 파일인지 추가가 되면 좋을 것 같습니다!

3. GraphRAG 구현

• GraphRAG가 어떻게 구현되는지 설명이 추가되면 좋겠습니다.
• neo4j의 Cypher를 저도 잘 모르고, 사용하는 사람도 모르니까 Cypher의 사용법이나 기초 문법(?) 같은 설명도 있으면 좋을 것 같아요.
• 지금 구현은 LangGraph를 안 사용하는 것 같은데, 추후에 Agent로 만드실 예정인지도 궁금합니다!

짧은 의견이지만, 가능하고 필요하다고 생각되는 파트만 적용해주시면 될 것 같습니다!
다음주에 완성되면 저도 neo4j로도 비교해서 돌려봐야겠습니다.. 제꺼는 너무 느려서요ㅠㅠ

@syshin0116 리뷰 최대한 반영해 보았습니다 (링크 추가나 아주 조금이지만 단점을 언급한 부분 등..?)
그리고 from_llm 부분은 큰 변화가 없어서 그냥 그대로 사용하였습니다! 리뷰에 많은 도움을 얻었습니다 감사합니다 (_ _)

@dancing-with-coffee 피드백주신 대로 데이터에 대한 설명 최대한 상세히 첨부하려 노력했습니다! 그리고 저도 논문을 추천하는 방향으로 가고 싶어 Langgraph를 활용해 보았는데... 성능이 영 안 좋아서 그대로 반영하는 게 맞을지 모르겠네요... ; - ;
아무튼 덕분에 작업물 퀄리티를 많이 높일 수 있었던 것 같습니다! 감사합니다(_ _)

[Todo]

아직 코드는 제대로 돌려보지 못해서, 외형적인 부분만 먼저 작성했습니다. 코드 실행 동작 확인 부분은 확인후에 체크하겠습니다!

[Review Checklist]

• Review OS:

  • Windows / Mac / Linux (Mac)

• Template Rule 준수 여부:

  • Template Rule 가이드를 준수하였는가? (Check 표기)

• Table of Contents 링크 확인:

  • Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)

Let's make the Academic Search System이 작동안합니다! let-s 에서 - 하나 빼니 작동하네요ㅎㅎ
[Let's make the Academic Search System](#lets-make-the-academic-search-system)

• 이미지 파일명 검토:

  • 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)

이미지 파일명이 10-academic-search-system 시작하는데 07-academic-search-system 로 수정이 필요해 보입니다!

• 최신 import 방식 사용 여부:

  • import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)

• 코드 동작 확인:

  • 모든 코드가 오류 없이 동작하는가? (Check 표기)
  • Warning 발생 시 코멘트에 적어주세요.

• 제출 파일 확인:

  • 꼭 필요한 파일들만 제출되었는가?

    • 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)

• 기타 의견:

Environement Setup

`NEO4J_URL` 대신 `NEO4J_URI`을 공통적으로 사용하기로 했습니다. 통일해주시면 좋을것 같습니다!

# Set environment variables
from langchain_opentutorial import set_env

set_env(
    {
        "OPENAI_API_KEY": "",
        "LANGCHAIN_API_KEY": "",
        "LANGCHAIN_TRACING_V2": "true",
        "LANGCHAIN_ENDPOINT": "https://api.smith.langchain.com",
        "LANGCHAIN_PROJECT": "Academic Search System", 
    }
)

이부분에 "NEO4J_URI", "NEO4J_USERNAME", "NEO4J_PASSWORD" 추가해주면 조금 더 친절한 튜토리얼이 될것 같아요!

docker-compose.yml 부분

```yaml : 3중 backtick 뒤에 yaml 명시해주면 색깔 들어가서 좀 더 보기 편해집니다!ㅎㅎ

LangGraph

그래프를 시각화해서 한번 보여주면 좋을것 같습니다

from IPython.display import Image, display
from langchain_core.runnables.graph import CurveStyle, MermaidDrawMethod, NodeStyles

# Visualize the compiled StateGraph as a Mermaid diagram
display(
    Image(
        app.get_graph().draw_mermaid_png(
            draw_method=MermaidDrawMethod.API,
        )
    )
)

고생하셨습니다!! 유용한 튜토리얼이어서, 조만간 참고해서 논문 검색해주는 서비스 만들어보고싶네요. Ta-da! 같은 텐션 재밋었습니다ㅋㅋㅋㅋ

피드백 반영하여 수정하였습니다!!! ㅎㅎㅎ 상세한 피드백 정말 감사드립니다🙇‍♂️

[Review Checklist]

• Review OS:

  • Windows / Mac / Linux (�Mac)

• Template Rule 준수 여부:

  • Template Rule 가이드를 준수하였는가? (Check 표기)

• Table of Contents 링크 확인:

  • Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)

• 이미지 파일명 검토:

  • 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)

• 최신 import 방식 사용 여부:

  • import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)

• 코드 동작 확인:

  • 모든 코드가 오류 없이 동작하는가? (Check 표기)
  • Warning 발생 시 코멘트에 적어주세요.

Environment Setup

langgraph 뒤에 "," 쉼표가 빠졌습니다

# Install required packages
from langchain_opentutorial import package

package.install(
    [
        "langchain-neo4j", 
        "langchain",
        "langchain_openai",
        "langchain_core",
        "langgraph"
        "pyalex",
    ],
    verbose=False,
    upgrade=False,
)

• 제출 파일 확인:

  • 꼭 필요한 파일들만 제출되었는가?
    • 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)

• 기타 의견:

def make_file

위에 json_path에 경로 넣고 끝에 "/'를 넣지 않는 경우엔 의도하신대로 작동하지 않는것 같습니다.
아래과 같이 수정하면 개선 가능할것 같아요

#수정 전
file = json_path + name.replace(" ", "_") + "_" + entity + ".json"

#수정 후
file = os.path.join(json_path, f"{name.replace(' ', '_')}_{entity}.json")

def relationship_vector_search_tool

사소하지만 "relathionship" 오타 발견했습니다ㅎㅎ

이외의 코드 실행은 문제 없었습니다.

리뷰하며 많이 배워갑니다ㅎㅎ고생 많으셨습니다!!

[Review Checklist]

• Review OS:

  • Windows / Mac / Linux (�Mac)

• Template Rule 준수 여부:

  • Template Rule 가이드를 준수하였는가? (Check 표기)

• Table of Contents 링크 확인:

  • Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)

• 이미지 파일명 검토:

  • 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)

• 최신 import 방식 사용 여부:

  • import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)

• 코드 동작 확인:

  • 모든 코드가 오류 없이 동작하는가? (Check 표기)
  • Warning 발생 시 코멘트에 적어주세요.

Environment Setup

langgraph 뒤에 "," 쉼표가 빠졌습니다

# Install required packages
from langchain_opentutorial import package

package.install(
    [
        "langchain-neo4j", 
        "langchain",
        "langchain_openai",
        "langchain_core",
        "langgraph"
        "pyalex",
    ],
    verbose=False,
    upgrade=False,
)

• 제출 파일 확인:

  • 꼭 필요한 파일들만 제출되었는가?

    • 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)

• 기타 의견:

def make_file

위에 json_path에 경로 넣고 끝에 "/'를 넣지 않는 경우엔 의도하신대로 작동하지 않는것 같습니다. 아래과 같이 수정하면 개선 가능할것 같아요

#수정 전
file = json_path + name.replace(" ", "_") + "_" + entity + ".json"

#수정 후
file = os.path.join(json_path, f"{name.replace(' ', '_')}_{entity}.json")

def relationship_vector_search_tool

사소하지만 "relathionship" 오타 발견했습니다ㅎㅎ

이외의 코드 실행은 문제 없었습니다.

리뷰하며 많이 배워갑니다ㅎㅎ고생 많으셨습니다!!

피드백 반영하였습니다 (_ _ ) 정말 감사합니다!