-
Notifications
You must be signed in to change notification settings - Fork 283
[E-1] 09-FewShotTemplates.ipynb #49
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
Conversation
[Review] 1. Review OS: Windows 2. Template Rule 가이드를 준수 하였는가: YES 3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES 4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES 5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES 6. 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): NO 7. 기타 의견: 코드 작성이 익숙치 않아 오류가 있을 수 있습니다. 알려주시면 수정하겠습니다.
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.
안녕하세요.
현재 파일이 3개가 포함되어있는데 모두 포함되어야 하는 파일일까요? 다시 한번 확인 부탁드립니다.
- 01-BASIC/05-Runnable.ipynb
- 01-BASIC/09-FewShotTemplates.ipynb
- 02-Prompt/[E-1] 09-FewShotTemplates.ipynb
- 디렉터리 경로 변경 필요 :
02-Prompt
>02-PROMPT
- 파일명 변경 필요 :
[E-1] 09-FewShotTemplates.ipynb
>09-FewShotTemplates.ipynb
- 디렉터리 경로 변경 필요 :
혹시 필요한 경우 아래의 링크를 참조하시면 도움이 될 것 같습니다.
- 필요가 없는 파일 삭제 : git rm
- 이름 변경 또는 파일 경로 변경 : 명령줄을 사용하여 파일 이름 바꾸기
관련하여 추가 문의 사항이나 의견이 있으시면 편하게 알려주세요.
잘못된 파일제거
폴더명이랑 파일명 수정하니 불필요 파일 2개가 사라진 거 같은데 맞나요...? |
넵, 현재 |
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]
- Review OS: Windows
- Template Rule 가이드를 준수 하였는가: YES
- Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES
- 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
- import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
- 모든 코드가 동작에 오류 없이 동작하는가?: YES
- 기타 의견:
- Table of Contents 중 FewShotPromptTemplate Example -> FewShotPromptTemplate 로 수정
- Author, Design 볼드 제거
- Peer review 추가
- "LANGCHAIN_PROJECT" 뒤 "05-Runnable" -> "09-Few-Shot Templates"
- 예제 South Korea -> United States of America
- 질문 :
- chain 생성에 LLMChain 클래스를 쓴 이유가 있는지?
- 제목 Few-Shot Templates -> Few-Shot Prompt Templates 어떨지?
- 파일 번호가 09가 아니라 02 아닐까요?
- FewShotChatMessagePromptTemplate 중. 예를 들어 회의록 작성하도록 예제를 넣는 부분에 회의록의 양식대로 되어 있지 않고 요약된 문장으로 들어가 있는 것 같아요.
- Resolving Similarity Search Issues in Example Selector 코드 부분 중복 확인 요청, print("Most similar example:")와 함께 question도 같이 출력되도록 하면 좋을 것 같습니다. 앞과 마찬가지로 회의록 양식 예제 확인 요함.
- 마지막 셀에 대한 질문. 예제로 입력된 answer가 그냥 출력되고 있는 것으로 이해. llm에 예제를 주고 양식에 맞춰 요약 또는 회의록 작성 또는 교정하도록 수정되어야 하는 것 아닌지요?
수정사항 반영 중(진행중)
수정사항 저장(진행중)
import 구문 수정 (langchain_openai) 출력 수정
[Review]