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

Skip to content

LangSmith Tracking Setup #28

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 13 commits into from
Jan 2, 2025
Merged

LangSmith Tracking Setup #28

merged 13 commits into from
Jan 2, 2025

Conversation

jeongkpa
Copy link
Contributor

[Title] LangSmith Tracking Setup
[Version] initial
[Language] ENG
[Packages] langsmith

-Setting up LangSmith trace configuration
-Implementing project-level tracking and monitoring -Integrating with Jupyter notebooks and Python code -Managing environment variables and API keys
-Using langchain-teddynote for simplified tracking

@jeongkpa
[Team] 기존1팀
8a793f3 
[Title] LangSmith Tracking Setup
[Version] initial
[Language] ENG
[Packages] langsmith

-Setting up LangSmith trace configuration
-Implementing project-level tracking and monitoring
-Integrating with Jupyter notebooks and Python code
-Managing environment variables and API keys
-Using langchain-teddynote for simplified tracking
@jeongkpa jeongkpa self-assigned this Dec 30, 2024
@jeongkpa jeongkpa changed the title [Team] 기존1팀 LangSmith Tracking Setup Dec 30, 2024
r14minji
r14minji requested changes Dec 31, 2024
View reviewed changes
Copy link
Contributor

@r14minji r14minji left a comment •
edited
Loading

Choose a reason for hiding this comment

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

[Review]

  1. Review OS(Windows/Mac/Linux): Window/ MAc
  2. Template Rule 가이드를 준수 하였는가(YES/NO): YES
  3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES
  4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): NO
  • 이미 제출된 파일이라 PASS 하면 될까요?
  • 이미지 파일명 가이드는 영문 소문자, 공백 대신 하이픈 - 으로 작성. 언더바 _로 작성된 것을 하이픈 - 으로 대체 해야합니다.
  • 양식은 jupyter notebook 파일명 + 이미지 제목 + 필요시 숫자
  1. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES
  2. 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES
  3. 기타 의견:
  • 렝체인노트(위키독스) 기준 LangSmith 의 추적기능에 관한 설명이 누락되었습니다. 의도하신 바 일까요? 혹시 놓치신 부분이라면 추가 요청드립니다.
  • copy_api_key_image.png 내에 "테스트 키" "Test key" 로 영문자로 변경 가능할까요?
  • 렝체인노트(위키독스) 기준 langchain-teddynote 하위 컨텐츠가 누락되었었습니다. 이 부분도 혹시 놓친 부분인지, 영어버전에서는 제외하기로 한 부분인지 확인 요청드립니다.

@jeongkpa
Copy link
Contributor Author

[Review]

  1. Review OS(Windows/Mac/Linux): Window/ MAc
  2. Template Rule 가이드를 준수 하였는가(YES/NO): YES
  3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?(YES/NO): YES
  4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?(YES/NO): NO
  • 이미 제출된 파일이라 PASS 하면 될까요?
  • 이미지 파일명 가이드는 영문 소문자, 공백 대신 하이픈 - 으로 작성. 언더바 _로 작성된 것을 하이픈 - 으로 대체 해야합니다.
  • 양식은 jupyter notebook 파일명 + 이미지 제목 + 필요시 숫자
  1. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?(YES/NO): YES
  2. 모든 코드가 동작에 오류 없이 동작하는가? (만약, warning 발생시 코멘트에 적어주세요): YES
  3. 기타 의견:
  • 렝체인노트(위키독스) 기준 LangSmith 의 추적기능에 관한 설명이 누락되었습니다. 의도하신 바 일까요? 혹시 놓치신 부분이라면 추가 요청드립니다.
  • copy_api_key_image.png 내에 "테스트 키" "Test key" 로 영문자로 변경 가능할까요?
  • 렝체인노트(위키독스) 기준 langchain-teddynote 하위 컨텐츠가 누락되었었습니다. 이 부분도 혹시 놓친 부분인지, 영어버전에서는 제외하기로 한 부분인지 확인 요청드립니다.

-> 이미지명 모두 변경 완료
-> 추적기능 추가
-> 랭체인 테디노트는 사용 X

@jeongkpa jeongkpa requested a review from r14minji December 31, 2024 09:50
jeongkpa and others added 6 commits January 1, 2025 11:34
@jeongkpa
Feat : add environment setup
7ddb017
@jeongkpa
fix : langsmith 오타 수정
361fff3
@jeongkpa
fix : 환경설정 순서 수정
6230246
@jeongkpa
Fix : 이미지 수정
6381e68
@jeong-wooseok
[Review]
1ba8638 
1. Review OS: Windows
2. Template Rule 가이드를 준수 하였는가: YES
3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES
4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
6. 모든 코드가 동작에 오류 없이 동작하는가? : Yes
7. 기타 의견:
- 설명이 전반적으로 잘 갖춰졌습니다. 이미지 수정되면, 한글부분이 없어질 것 같습니다.
@jeong-wooseok
Merge branch 'main' of https://github.com/jeongkpa/LangChain-OpenTuto…
297fced 
…rial into pr/28
jeong-wooseok
jeong-wooseok previously approved these changes Jan 1, 2025
View reviewed changes
Copy link
Contributor

@jeong-wooseok jeong-wooseok left a comment

Choose a reason for hiding this comment

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

[Review]

  1. Review OS: Windows
  2. Template Rule 가이드를 준수 하였는가: YES
  3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES
  4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
  5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
  6. 모든 코드가 동작에 오류 없이 동작하는가? : Yes
  7. 기타 의견:
  • 확인 완료하였습니다

@jeongkpa jeongkpa added the docs tutorial label Jan 1, 2025
@jeong-wooseok
[Review]
b806f72 
1. Review OS: Windows
2. Template Rule 가이드를 준수 하였는가: YES
3. Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES
4. 이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
5. import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
6. 모든 코드가 동작에 오류 없이 동작하는가? : Yes
7. 기타 의견:
- 이미지이름의 파일링크로 수정하여 맞췄습니다.
@r14minji
Copy link
Contributor

r14minji commented Jan 1, 2025

[Review] 수정사항 확인

Review OS: Mac
Template Rule 가이드를 준수 하였는가: YES
Table of Contents 의 링크가 원활하게 동작하는지 확인하였는가?: YES
이미지가 포함되어 있다면, 이미지의 파일명이 가이드를 준수하였는가?: YES
import 구문이 예전 legacy 방식이 아닌 최신 버전을 따르는가?: YES
모든 코드가 동작에 오류 없이 동작하는가? : Yes
기타 의견:
확인 완료하였습니다, approve 진행하겠습니다

@teddylee777
Copy link
Contributor

@jeong-wooseok 님 리뷰 감사드립니다.
추후 리뷰시 문제 없다면 approve 로 체크해 주시면 우측 상단에 approve 마킹이 됩니다~ 다음에는 체크 부탁 드려요~

@teddylee777 teddylee777 merged commit 5c20d5f into LangChain-OpenTutorial:main Jan 2, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@r14minji r14minji r14minji approved these changes

@jeong-wooseok jeong-wooseok jeong-wooseok left review comments

Assignees

@jeongkpa jeongkpa

Labels
docs tutorial
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jeongkpa @r14minji @teddylee777 @jeong-wooseok