search

fgenie / rims_minimal

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

added safety lines to get_concordant_answer() #29

Closed fgenie closed 5 months ago

fgenie commented 5 months ago

prevent wrong processing of "None"

get_concordant_answer() 에 "None" (스트링) 이 들어가는 경우는 dd4c82a (_convert_to_str_if_not_none_nor_float() permalink) 이후 해결되었으나, 지금 나와같이 그것보다 이전 코드의 아웃풋을 재사용하는 경우 발생할 수 있는 케이스에 대한 추가처리 로직을 추가

MATH chatgpt 결과에서만 이 문제가 보여길래 찾아봤더니 이랬다

해당 수정 후에 #20 의 마지막 수정 결과와 같은 점수가 나오는 것을 통해서 검증됨.

fgenie commented 5 months ago

@seanexp 코드변경에 대한 트래킹이 필요할 것 같아서 PR했었구요... MATH가 일단락 된 상황에서 리뷰를 꼭 받지 않아도 될 수도 있겠네요. 히스토리 목적으로 남겨둘게요. 머지하고 닫습니다.