-
Notifications
You must be signed in to change notification settings - Fork 282
[N-1] 03-GraphDB / 05-TitanicQASystem #249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[N-1] 03-GraphDB / 05-TitanicQASystem #249
Conversation
Initial commit (초안)
feat: Add content details and Neo4j import process
[TODO]
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
안녕하세요.
CSV 데이터 셋을 활용한 관계 분석 및 QA 아이디어가 너무 좋네요.
데이터셋 다운로드 기능 및 데이터 셋에 대한 상세한 설명이 추가되어 있어 이해하기 쉬웠습니다.
차주에는 예시를 추가하여 마무리 부탁 드립니다.
수고하셨습니다!
안녕하세요. 설명도 잘 되어있고 데이터셋도 익숙해서 친절한 튜토리얼이 될 듯 합니다! |
feat : contents 추가
[TODO]
|
fix : langgraph workflow 에러 수정
fix : langgraph workflow , neo4j 사진첨부, 번역, 포맷수정
fix : ipynb 파일 format 맞춰 수정
fix : 오타수정
2차커밋 완료했습니다~! pr 부탁드립니다! [Note] pr 해주시는 @greencode-99 @MoonHeesun 두분도 제목부분 어떤지 의견주시면 감사할거같습니다! |
There was a problem hiding this 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 기반으로 예시까지 제공하여 많은 분들이 참고할 수 있겠네요.
아래 몇 가지 수정 요청 드립니다.
- assets, data 경로 수정
- .ipynb 기준으로 동일 위치에 data, assets 폴더 생성
-
오탈자 수정 : Cypher queryr -> Cypher query
text2cypher_prompt = ChatPromptTemplate.from_messages(
...
Cypher queryr: "MATCH (p:Passenger {{name: "John
...
) -
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
@greencode-99 |
@Taylor0819 제가 저 부분을 못보고 넘어갔네요. |
fix : image 경로 & assets, data 디렉토리 재배치 및 코드수정 file name 수정 (pm님과 합의 FAQRAGSystem → TitanicQASystem) peer review 1 수정사항 반영 (오탈자, 기타파일 경로 및 재배치)
@back2zion 파일 이름 TitanicQASystem으로 수정했습니다~~! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Taylor0819
변경사항 모두 정상 확인 하였고,
Peer Review 항목만 작성 요청 드립니다.
- Author: Taylor(Jihyun Kim)
- Design:
- Peer Review:
- This is a part of LangChain Open Tutorial
수고하셨습니다!
feat : peer review 1 완료
종철님 추가 완료하였습니다 ~~!! |
fix : image file 경로 수정
[Review Checklist]
꼼꼼히 잘 작성해주신 것 같습니다. 고생 많으셨습니다! |
There was a problem hiding this 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 추가 <추가> 이미지 노드 컬러 수정
@MoonHeesun @greencode-99 방금 수정사항 반영해서 다시 커밋했습니다~! 확인해주시면 approve 부탁드립니다! ¨̮ |
fix : file # 제목 수정 (05-TitanicQASystem → TitanicQASystem)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2주동안 수고 많으셨습니다!
너무 고생하셨습니다! |
82e78f6
into
LangChain-OpenTutorial:main
{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)
If no one reviews your PR within a few days, please @-mention one of teddylee777, musangk, BAEM1N