Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Taylor0819
Copy link
Contributor

@Taylor0819 Taylor0819 commented Jan 8, 2025

  • Initial commit (초안) titanic 데이터로 관계에 대한 분석 (자연어 → cypher) 입니다!
  • 0115 check list 확인 후 pr 요청

{PR Message - Write freely, remove this bracket if unnecessary}

Author Checklist

  • PR Title Format: I have confirmed that the PR title follows the correct format. (e.g., [N-2] 07-Text Splitter / 07-RecursiveCharacterTextSplitter)

  • Committed Files: I have ensured that no unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) are included. These files are not allowed.

  • (Optional) Related Issue: If this PR is linked to an issue, I have referenced the issue number in the PR message. (e.g., Fixes Update 01-PromptTemplate.ipynb #123)

  • ❌ Do not include unnecessary files (e.g., .bin, .gitignore, poetry.lock, pyproject.toml) or other people's code. If included, close the PR and create a new PR.

Review Template (Intial PR)

🖥️ OS: Mac
✅ Checklist      
 - [x] **Template**: Tutorials follows the required template. 
 - [x] **Table of Contents(TOC) Links**: All Table of Contents links work. ((Yes/No)
 - [x] **Image**: Image filenames follow guidelines.
 - [x] **Imports*: All import statements use the latest versions. Ensure "langchain-teddynote" is not used. 
 - [x] **Code Execution**: Code runs without errors.
 - Comments: 어쩌다보니 제 파일이 좀 많이 길어졌습니다 ㅠㅠ 두분 고생하시겠지만 pr 부탁드립니다!  제가 두분 pr은 더 꼼꼼히 챙겨보겠습니다!!

If no one reviews your PR within a few days, please @-mention one of teddylee777, musangk, BAEM1N

Initial commit (초안)
@Taylor0819 Taylor0819 self-assigned this Jan 9, 2025
feat: Add content details and Neo4j import process
@Taylor0819
Copy link
Contributor Author

Taylor0819 commented Jan 9, 2025

[TODO]

  • neo4j desktop set up 앞부분 링크걸기
  • 콘텐츠 내용 채우기 (자연어 → cypher)
  • 작성 format 맞춰서 수정
  • flow 재확인
  • graph db 그룹 table of contents 겹치는부분 통일 멘트나 단어 통일하기
  • 한국어 → 영어

Copy link
Contributor

@greencode-99 greencode-99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요.
CSV 데이터 셋을 활용한 관계 분석 및 QA 아이디어가 너무 좋네요.
데이터셋 다운로드 기능 및 데이터 셋에 대한 상세한 설명이 추가되어 있어 이해하기 쉬웠습니다.
차주에는 예시를 추가하여 마무리 부탁 드립니다.

수고하셨습니다!

@MoonHeesun
Copy link
Contributor

MoonHeesun commented Jan 9, 2025

안녕하세요. 설명도 잘 되어있고 데이터셋도 익숙해서 친절한 튜토리얼이 될 듯 합니다!
예제까지 추가되면 더욱 완벽해질 것 같네요 :)
2차 때 review checklist에 따라 다시 확인하도록 하겠습니다.
고생 많으셨습니다!

feat : contents 추가
@Taylor0819
Copy link
Contributor Author

Taylor0819 commented Jan 12, 2025

[TODO]

  • neo4j desktop set up 앞부분 링크걸기
  • 작성 format 맞춰서 수정
  • flow 재확인
  • 한국어 → 영어
  • 랭그래프 flow 수정
  • neo4j 시각화 export 및 첨부
  • data import 사진 수정 (file 이름 다름)
  • output prettyprint로 수정

fix : langgraph workflow 에러 수정
fix : langgraph workflow ,
neo4j 사진첨부, 번역, 포맷수정
fix : ipynb 파일 format 맞춰 수정
@Taylor0819
Copy link
Contributor Author

2차커밋 완료했습니다~! pr 부탁드립니다!

[Note]
@back2zion pm님 많이 바쁘시겠지만 제 파일이 FAQRAGSystem 쪽이랑은 살짝 멀어진거같아서 TitanicRelationshipQASystem 이나 TitanicQASystem 정도로 수정하는게 어떨까싶은데 한번 내용 훑어봐주시고 확인부탁드립니다!

pr 해주시는 @greencode-99 @MoonHeesun 두분도 제목부분 어떤지 의견주시면 감사할거같습니다!

Copy link
Contributor

@greencode-99 greencode-99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Review OS:

    • Mac
  • Template Rule 준수 여부:

    • Template Rule 가이드를 준수하였는가? (Check 표기)
  • Table of Contents 링크 확인:

    • Table of Contents의 링크가 원활하게 동작하는지 확인하였는가? (Check 표기)
  • 이미지 파일명 검토:

    • 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가? (Check 표기)
  • 최신 import 방식 사용 여부:

    • import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가? (Check 표기)
  • 코드 동작 확인:

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

    • 꼭 필요한 파일들만 제출되었는가?
    • 튜토리얼 외 설정 파일이나 다른 분이 작업한 파일이 포함되었는지 Files Changed에서 확인 부탁드립니다. (Check 표기)
  • 기타 의견:
    수고 많으셨습니다. LangGraph 기반으로 예시까지 제공하여 많은 분들이 참고할 수 있겠네요.
    아래 몇 가지 수정 요청 드립니다.

  1. assets, data 경로 수정
  • .ipynb 기준으로 동일 위치에 data, assets 폴더 생성
  1. 오탈자 수정 : Cypher queryr -> Cypher query
    text2cypher_prompt = ChatPromptTemplate.from_messages(
    ...
    Cypher queryr: "MATCH (p:Passenger {{name: "John
    ...
    )

  2. APOC Plug-in 설치 가이드 추가 필요

  • plug-in 설치가 필요하다는 안내 문구 또는 가이드가 없어서 무작정 따라하다보니 아래와 같이 오류가 발생 했습니다.
    Connection error: Could not use APOC procedures. Please ensure the APOC plugin is installed in Neo4j and that 'apoc.meta.data()' is allowed in Neo4j configuration

@Taylor0819
Copy link
Contributor Author

Taylor0819 commented Jan 16, 2025

@greencode-99
안녕하세요 종철님!
와 오탈자 어떻게 찾으셨어요.. 대박 감사합니다 ㅋㅋㅋ
제가 앞부분이 희선님과 neo4j desktop 세팅이동일해서 테디님과 상의해서 링크거는걸로 해두었거든요 ..!
링크따라서 apoc pulgin 설치와 conf 파일 수정하라고 써놓긴했는데..
혹시 잘안보이면 더 잘보이게끔 수정을 해볼까요?
image

@greencode-99
Copy link
Contributor

greencode-99 commented Jan 16, 2025

@Taylor0819 제가 저 부분을 못보고 넘어갔네요.
링크와 안내가 추가되어 있으니 별도의 조치는 안하셔도 되겠습니다.
그리고, Peer Review 항목 추가 요청 드립니다:)

fix : image 경로 & assets, data 디렉토리 재배치 및 코드수정
file name 수정 (pm님과 합의 FAQRAGSystem → TitanicQASystem)
peer review 1 수정사항 반영 (오탈자, 기타파일 경로 및 재배치)
@Taylor0819
Copy link
Contributor Author

@back2zion 파일 이름 TitanicQASystem으로 수정했습니다~~!
@greencode-99 @MoonHeesun 참고 부탁드립니다 ¨̮

@Taylor0819 Taylor0819 changed the title [N-1] 03-GraphDB / 05-FAQRAGSystem [N-1] 03-GraphDB / 05-TitanicQASystem Jan 16, 2025
@Taylor0819 Taylor0819 mentioned this pull request Jan 16, 2025
Copy link
Contributor

@greencode-99 greencode-99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Taylor0819
변경사항 모두 정상 확인 하였고,
Peer Review 항목만 작성 요청 드립니다.

수고하셨습니다!

feat : peer review 1 완료
@Taylor0819
Copy link
Contributor Author

Taylor0819 commented Jan 17, 2025

종철님 추가 완료하였습니다 ~~!!

fix : image file 경로 수정
@MoonHeesun
Copy link
Contributor

[Review Checklist]

  • Review OS:

    • Windows
  • Template Rule 준수 여부:

    • Template Rule 가이드를 준수하였는가?
  • Table of Contents 링크 확인:

    • Table of Contents의 링크가 원활하게 동작하는지 확인하였는가?
  • 이미지 파일명 검토:

    • 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?
  • 최신 import 방식 사용 여부:

    • import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?
  • 코드 동작 확인:

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

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

  1. Open in GitHub 뱃지 링크와 03-GraphDB/04-MovieQASystem 링크 오탈자 수정

    • ~/9-Cookbook/~~/19-Cookbook/~
  2. 아래 부분에서 Cypher 쿼리 용어는 백틱으로 강조하면 어떨까요? 가독성이 더 좋아질 것 같아 제안드립니다 :)

    image

    image

꼼꼼히 잘 작성해주신 것 같습니다. 고생 많으셨습니다!

greencode-99
greencode-99 previously approved these changes Jan 17, 2025
Copy link
Contributor

@greencode-99 greencode-99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

완벽하네요~ 수고하셨습니다🙏😄

fix : Peer review2 반영완료
- 링크 오타 수정
- cypher 관련 backtick 추가
<추가>
이미지 노드 컬러 수정
@Taylor0819
Copy link
Contributor Author

@MoonHeesun @greencode-99 방금 수정사항 반영해서 다시 커밋했습니다~! 확인해주시면 approve 부탁드립니다! ¨̮

fix : file # 제목 수정 (05-TitanicQASystem → TitanicQASystem)
Copy link
Contributor

@greencode-99 greencode-99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2주동안 수고 많으셨습니다!

@MoonHeesun
Copy link
Contributor

너무 고생하셨습니다!

@teddylee777 teddylee777 merged commit 82e78f6 into LangChain-OpenTutorial:main Jan 18, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants