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

Skip to content

Conversation

Jae-hoya
Copy link
Contributor

@Jae-hoya Jae-hoya commented Jan 5, 2025

{PR Message - Write freely, remove this bracket if unnecessary}
Description of the issue & Suggested Fix or Improvement
참고링크

오탈자 수정

Table of Contents 쪽 - [Environement Setup]~에서 Environement를
Environment로 수정 -> O

!pip install langchain-opentutorial를
%pip install langchain-opentutorial로 수정
*참고: 앞으로도 ! 대신 %로 통일 할 것 같습니다 수정 -> O

-You can checkout the [langchain-opentutorial]~ 에서 checkout 를
check out로 수정 수정 -> O

마크다운 형식
.env를 .env로 수정 수정 -> O

문법 교정

For example, you can return instructions as a string that describe the fields ~ 에서 describe를
describes로 수정 수정 -> O

easy-to-use environment setup, useful에서 setup를
setups로 수정 수정 -> O

'functions' 뒤에 쉼표 추가: useful functions and utilities → useful functions, and utilities (크게 신경쓸만한 포인트는 아닌 것 같지만 한번 수정해봤습니다.) 수정 -> O

'such as' 앞에 쉼표 추가: "API keys such as OPENAI_API_KEY" → "API keys, such as OPENAI_API_KEY" 수정 -> O

'an'삭제: When provided with an email content → When provided with email content 수정 -> O

+) 수정하시고 리뷰어 걸어주실 때, 저랑 규식(Q0211)님 걸어주시면 됩니다!

이미 MR되있는 파일을, 리뷰어님들의 교정을 받아 수정하였습니다.
추가적으로 제목에 오탈자가 있어서 수정하였습니다.
01-PydanticOuputParser -> 01-PydanticOutputParser

고생해주신 보경님, 규식님 감사드립니다.

Author Checklist

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

  • [O ] 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.

Jae-hoya and others added 3 commits January 5, 2025 23:59
BokyungisaGod
BokyungisaGod previously approved these changes Jan 6, 2025
@teddylee777 teddylee777 added the proofreading 번역/검수팀 제안사항 반영 label Jan 6, 2025
@teddylee777 teddylee777 changed the title [E-2] / 03-OutputParser / 01-PydanticOutputParser [E-2] 03-OutputParser / 01-PydanticOutputParser Jan 7, 2025
Jae-hoya and others added 2 commits January 8, 2025 20:11
[Title] 02-RAG-Advanced.ipynb
[Version] initial
[Language] ENG
[Packgage] "bs4", "faiss-cpu", "pypdf", "unstructured", "unstructured[pdf]", "chromadb", "rank_bm25", "langsmith", "langchain", "langchain_text_splitters", "langchain_community", "langchain_core", "langchain_openai", "langchain_experimental"
.
 이전 pr이 mr되지 않아 겹치지 않아 작업 완료된 파일을 뻇습니다.
@teddylee777
Copy link
Contributor

[PR Open] 불가 이슈 확인으로 잠시 닫겠습니다. (다시 열도록 하겠습니다.)

@teddylee777 teddylee777 closed this Jan 8, 2025
@Q0211
Copy link

Q0211 commented Jan 8, 2025

  1. Pydantic"Ouput"Parser를 PydanticOutputParser로 수정해야할 것 같습니다.
  2. llm_with_structered를 llm_with_structured로 바꿔야 할 것 같습니다. 제가 찾은 바로는 2군데 오타가 있었습니다.
  3. Below is an example of email text을 Below is an example of an email conversation stored in the variable email_conversation 로 바꾸는 것이 더 좋아보입니다.
  4. Inputs the content of the email content를 Inputs the content of the email conversation로 바꾸는 것이 더 좋아보입니다
  5. 제목 중 create chain with parser에서 Create와 같이 대문자를 써주셔야 할 것 같습니다.
  6. The result is output in JSON format을 The result is provided in JSON format이라고 하는 것이 더 직관적이지 않을까 합니다.

@Jae-hoya
Copy link
Contributor Author

Jae-hoya commented Jan 8, 2025

수정사항들 모두 수정하였습니다. 기존 PR이 끝나면 재PR하도록 하겠습니다.

@chaeyoonyunakim chaeyoonyunakim added the bug Something isn't working e.g., PR conflict label Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working e.g., PR conflict proofreading 번역/검수팀 제안사항 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants