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

Skip to content

Conversation

MoonHeesun
Copy link
Contributor

@MoonHeesun MoonHeesun commented Jan 8, 2025

draft code

{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: Win/Mac/Linux   
✅ Checklist      
 - [ ] **Template**: Tutorials follows the required template. 
 - [ ] **Table of Contents(TOC) Links**: All Table of Contents links work. ((Yes/No)
 - [ ] **Image**: Image filenames follow guidelines.
 - [ ] **Imports*: All import statements use the latest versions. Ensure "langchain-teddynote" is not used. 
 - [ ] **Code Execution**: Code runs without errors.
 - Comments: {Write freely, 한국어 기술 가능}     

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

@MoonHeesun
Copy link
Contributor Author

현재까지 작성한 drft code는 기본적인 예제에 설명을 추가한 상태입니다. 이후부터는 kaggle movie dataset을 이용하여 영화 추천 예제까지 추가하여 더 수정할 생각입니다!

@Taylor0819
Copy link
Contributor

안녕하세요! 플로우, 내용적으로 아주좋습니다!
examples 가 더 있었으면 했는데 마침 추가한다고 하셔서 ㅎㅎ 그거까지 추가되면 더 좋을거같아요!
기타 포맷이나 준수사항은 다음주 리뷰때 더 꼼꼼히 보겠습니다 고생하셨습니다~~!!

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.

안녕하세요.
neo4j 설치 및 데이터 셋 사용 방법에 대한 가이드가 상세히 작성되어 너무 좋았습니다.
다음주까지 잘 마무리 부탁 드리고, 차주에 check list 기반으로 확인 하도록 하겠습니다.

수고하셨습니다:)

@greencode-99
Copy link
Contributor

greencode-99 commented Jan 16, 2025

  • 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 표기)
  • 기타 의견:
    신규 튜토리얼을 작성하시느라 수고 많으셨습니다.
    아래 2가지만 확인 후 수정 요청 드립니다.

    1. Peer Review 추가 필요
    2. 한국어 주석 수정 또는 삭제
    • os.environ["NEO4J_PASSWORD"] = "movie1234" # 초기 설정으로 바꾸기기

@Taylor0819
Copy link
Contributor

🖥️ OS: Mac
✅ Checklist

  • Template: Tutorials follows the required template.
  • Table of Contents(TOC) Links: All Table of Contents links work. ((Yes)
  • Image: Image filenames follow guidelines.
  • *Imports: All import statements use the latest versions. Ensure "langchain-teddynote" is not used.
  • Code Execution: Code runs without errors.
  • Comments: {Write freely, 한국어 기술 가능}
  1. 경로 수정 + 이미지 파일, 디렉토리 재배치
    테디님께 문의하여 ipynb 파일 기준으로 같은 디렉토리에 assets, data 디렉토리 생성
    “./assets/“ 으로 경로 수정 및 이미지 파일, 디렉토리 재배치
It explains the process of storing data in a graph database and impl
  1. Output 상이
    제 실행 코드에서 아웃풋과 희선님 파일의 아웃풋이 상이합니다! 

LOAD CSV WITH HEADERS FROM
    'https://raw.githubusercontent.com/tomasonjo/blog-datasets/main/movies/movies_small.csv'
    AS row
    MERGE (m:Movie {id:row.movieId})
    SET m.released = date(row.released),
    m.title = row.title,
    m.imdbRating = toFloat(row.imdbRating)
    FOREACH (director in split(row.director, '|') |
    MERGE (p:Person {name:trim(director)})
    MERGE (p)-[:DIRECTED]->(m))
    FOREACH (actor in split(row.actors, '|') |
    MERGE (p:Person {name:trim(actor)})
    MERGE (p)-[:ACTED_IN]->(m))
    FOREACH (genre in split(row.genres, '|') |
    MERGE (g:Genre {name:trim(genre)})
    MERGE (m)-[:IN_GENRE]->(g))
Graph Schema Pasted Graphic 7

Cypher 에서도 relationship은 제 아웃풋과 동일한거같은데 확인부탁드립니다! 혹 제가 잘못 실행했을까용..?

<희선님 output>


Pasted Graphic 10
Received notification


3. kaggle 데이터셋 example 이 쿼리 몇개 더 있으면 좋을거같습니다~!

고생많으셨습니다!!

@MoonHeesun
Copy link
Contributor Author

MoonHeesun commented Jan 18, 2025

  • Peer Review 추가 필요

  • 한국어 주석 수정 또는 삭제

    • os.environ["NEO4J_PASSWORD"] = "movie1234" # 초기 설정으로 바꾸기기
  • 경로 수정 + 이미지 파일, 디렉토리 재배치

  • Output 상이

    • 저의 경우, 기존 샘플 movie 데이터셋을 지우지 않은 채로 새로운 movies_small.csv 데이터셋을 import 해서 관계가 더 많이 정의된 것 같습니다. 이부분은 기존 movie 데이터셋을 지우고 새로운 데이터셋을 import 해오는 것으로 수정하였습니다!
  • kaggle 데이터셋 example 이 쿼리 몇개 더 있으면 좋을거같습니다~!

    • 현재 쿼리 추가 중 연결에 문제가 생겨 중단한 상태입니다. 일정이 있어 바로 수정하지 못하는 상황입니다..ㅠㅠ 해당 부분은 저녁 이후로 수정 가능할 것 같습니다. 계속 늦어지는 점 다시 한 번 사과드립니다...

greencode-99
greencode-99 previously approved these changes Jan 18, 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
Copy link
Contributor

아웅 괜찮습니다! 내일 전 일정이없어서 수정해주시고 말씀해주심 바로 확인하고 승인해드리겠습니다! 고생하셨습니다!!

@Taylor0819
Copy link
Contributor

희선님 혹시 수정하시면 디코나 여기에 멘션 한번해주심 바로 확인해드리겠습니다~~!

@MoonHeesun
Copy link
Contributor Author

@Taylor0819 지현님 늦은 시간에 죄송합니다..!
방금 예제 추가하였습니다. 간단한 수정임에도 계속 늦어진 점 죄송합니다 ㅠㅠ
감사합니다!

Copy link
Contributor

@Taylor0819 Taylor0819 left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!!!

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.

수고하셨습니다:)

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

Successfully merging this pull request may close these issues.

5 participants