fgenie / rims_minimal

Been lazy enough to pull over again to the end!
0 stars 1 forks source link

pqdm now + gpt4turbo causes IndexError (not happened before) #19

Closed fgenie closed 5 months ago

fgenie commented 5 months ago

chatgpt를 사용할때는 문제가 없는데 gpt4turbo를 사용하는경우 IndexError가 발생합니다.

그래서 pqdm 변경에 따른 문제라고 보고 있기는 한데... 봐주시겠어요? src/new 안에 스크립트, 그리고 4turbo 결과 파일이 첨부되어있습니다. 에러는 *.jsonl.error에 기록되어있습니다.

seanexp commented 5 months ago

이건 이슈로 올리셔야 할 거 같네요.

.jsonl 파일이나 .errors 파일을 커밋하시면 코드 관리가 어려워집니다. 뭐가 중요한 파일인지 보이지 않아서요

seanexp commented 5 months ago

그리고 tqdm 에서는 문제없이 돌아가는지 한번 확인 부탁드려요

fgenie commented 5 months ago

파일은 그럼 하던대로 슬랙에만 올릴게요 스크립트, 읽는 파일, 나온 파일 같이 확인해달라고 결과물을 함께 푸시했어요.

fgenie commented 5 months ago

tqdm은 확인해보겠습니다. 퇴근하고 공유해드릴게요

fgenie commented 5 months ago

pqdm문제 아니고 파싱문제인거만 확인했습니다 퇴근해서 마저 수정하고 푸시할게요

fgenie commented 5 months ago

추가로... temperature 0에 seed 를 줘도 output이 유지되진 않는 것 같네요. 결과가 큰 폭으로 변하진 않지만 디버깅할 때는 고려해야할 것 같습니다.

seanexp commented 5 months ago

파일은 그럼 하던대로 슬랙에만 올릴게요 스크립트, 읽는 파일, 나온 파일 같이 확인해달라고 결과물을 함께 푸시했어요.

파일을 깃헙 이슈에 올리시는거 낫지 않을까요? 슬랙은 파일 공유하면 다른 대화에 묻히는 경향이 있어서 ㅠ

뭐 편하신대로 하셔도 될거 같습니다.

seanexp commented 5 months ago

일단 이 이슈는 close 할테니 새로 이슈 열어주시면 좋을거 같습니다.