Skip to content

fix: 키 상세의 모델 행도 금액 기준으로 가른다 - #117

Merged
yessjun merged 1 commit into
mainfrom
fix/key-model-rows-split-by-price
Sep 7, 2026
Merged

fix: 키 상세의 모델 행도 금액 기준으로 가른다#117
yessjun merged 1 commit into
mainfrom
fix/key-model-rows-split-by-price

Conversation

@yessjun

@yessjun yessjun commented Sep 7, 2026

Copy link
Copy Markdown
Member

📝 작업내용

  • LlmKeyModelUsage에 가격이 붙은 입력 토큰과 출력 토큰, 두 평균 응답, 실패 수 추가
  • 원본 이벤트 질의에 같은 필터 다섯 추가. 이 조회는 일별 집계가 아니라 원본을 읽으므로 마이그레이션 없음
  • API 명세 v0.70.0. 연산 수 변화 없음

⭐️ 검증

  • mvn clean test 1343건 통과, 실패와 오류 0
  • 변이로 빨강 확인. 금액이 안 붙은 쪽 평균 응답의 필터를 붙은 쪽으로 뒤집으면 빨강. 되돌린 뒤 트리에 남은 것이 의도한 다섯 파일뿐인 것을 확인
  • 픽스처를 먼저 구별 가능하게 고쳤다. 종전에는 가격이 붙은 이벤트와 안 붙은 이벤트의 토큰이 5와 0으로 같고 지연도 둘 다 1이라, 두 평균을 맞바꿔도 토큰 합을 뒤집어도 아무 단언이 안 움직였다
  • 인접 라운드가 먼저 병합한 0.69.0 위로 리베이스하고 명세를 다시 뽑았다. 그 파일은 손으로 병합하면 안 되는 산출물이라 되돌리고 재생성

💬 리뷰 포인트

  • 키 상세만 맞추고 소비처와 호출 종류 표는 그대로 둔 것. 그 둘은 모델 여럿을 묶으므로 같은 방식으로 나누면 금액이라는 것이 아예 없는 자체 서빙 요청이 「모르는 쪽」에 통째로 들어간다. 한 화면의 세 표가 다른 규칙을 쓰는 대가가 있다

The key screen drew a model as one row with an amount beside the whole
request count, and that amount covered only the requests a provider had
priced. Eighteen of eighty-seven, said as "87회 · $0.079494". The platform
breakdown already splits the same model into the part with an amount and
the part without, so the same fact read two ways depending on which screen
was open, and the screen that said less was the one that invited the wrong
reading.

The five fields the platform row carries now travel on the key row as well.
This side reads raw events rather than the rollup, so they are filters
rather than columns and there is no migration.
@yessjun
yessjun merged commit ccf06ac into main Sep 7, 2026
1 check passed
@yessjun
yessjun deleted the fix/key-model-rows-split-by-price branch September 7, 2026 07:34
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