Skip to content
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

[COZY-243] feat: 모델 변경 #104

Merged
merged 1 commit into from
Sep 30, 2024
Merged

[COZY-243] feat: 모델 변경 #104

merged 1 commit into from
Sep 30, 2024

Conversation

eple0329
Copy link
Member

#️⃣ 요약 설명

Sonnet에서 Haiku로 수정입니다.
(코드 길이가 길어서 토큰 소비가 많을 것 같기에..)

📝 작업 내용

ex) 코드의 흐름이나 중요한 부분을 작성해주세요.
ex) 기존 calculate 함수의 버그를 수정했습니다.

중요한 코드 부분을 붙여넣어 설명해주세요.

동작 확인

ex) 기능을 실행했을 때 정상 동작하는지 여부를 확인하고 사진을 올려주세요
ex) 테스트 코드 작성도 좋습니다!

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@eple0329
Copy link
Member Author

[REVIEW]

안녕하세요 한국 Assistant입니다. 코드 리뷰를 자세히 진행하겠습니다.

1. .github/scripts/pr_review.py 파일에 대한 리뷰

1.1. 모델 ID 변경

  • 문제: 'anthropic.claude-3-5-sonnet-20240620-v1:0' 모델 ID가 적절하지 않습니다.
  • 개선 사항: 'anthropic.claude-3-haiku-20240307-v1:0' 모델 ID로 변경하는 것이 더 적절해 보입니다. 이 모델은 보다 간결한 형식의 텍스트 생성에 적합할 것 같습니다.
response = bedrock.converse(
    modelId='anthropic.claude-3-haiku-20240307-v1:0',  # 또는 다른 적절한 모델
    messages=[
        {
            'role': 'user',
            'content': formatted_prompt
        }
    ]
)

1.2. 코드 리뷰 포맷 개선

  • 문제: 코드 리뷰 포맷이 다소 복잡해 보입니다.
  • 개선 사항: 리뷰 포맷을 좀 더 간단하게 만들 수 있습니다. 예를 들어 다음과 같이 정리할 수 있습니다:
# Code Review
## 1. [파일명] 리뷰
### 1.1. [이슈 제목]
- 문제: [문제 설명]
- 제안: [개선 사항]

## 2. [다른 파일명] 리뷰
### 2.1. [이슈 제목]
- 문제: [문제 설명]
- 제안: [개선 사항]

이렇게 하면 리뷰 내용을 더 명확하게 전달할 수 있습니다.

전반적으로 코드 변경 사항은 적절해 보이며, 모델 ID와 리뷰 포맷 개선을 통해 코드 리뷰 프로세스를 개선할 수 있을 것 같습니다. 좋은 작업 감사드립니다.

@eple0329 eple0329 merged commit 6ff46a0 into develop Sep 30, 2024
1 check passed
@eple0329 eple0329 deleted the feature/COZY-243 branch September 30, 2024 02:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant