Skip to content

data(intent): 검증된 final data 및 split, model 가중치 추가 - #41

Merged
chaeeunn4 merged 2 commits into
mainfrom
docs/add-data-and-models
Aug 6, 2026
Merged

data(intent): 검증된 final data 및 split, model 가중치 추가#41
chaeeunn4 merged 2 commits into
mainfrom
docs/add-data-and-models

Conversation

@chaeeunn4

Copy link
Copy Markdown
Contributor

연결 Issue

Closes #

변경 내용

  • intent final 데이터 추가
  • intent data split 기준 id 추가
  • model 학습 후 산출물 ( 가중치 ) 추가
  • read me에 api 주소 추가

변경 이유

  • main 브랜치에 없는 정보 추가합니다.

검증

  • PR 제목이 <type>(<scope>): <한국어 요약> 형식입니다.
  • make validate
  • make lint
  • make test
  • 공식 출처와 확인일 검토
  • 개인정보·민감정보 미포함 확인

데이터 변경

항목 내용
추가/수정/삭제 건수 해당 없음
영향받는 Intent/Workflow 해당 없음
출처 해당 없음
교차 검수자 해당 없음

영향 및 제한사항

리뷰 포인트

@BcKmini
BcKmini self-requested a review August 6, 2026 07:39
@BcKmini

BcKmini commented Aug 6, 2026

Copy link
Copy Markdown
Member

LGTM

Comment thread .gitattributes
@@ -0,0 +1 @@
fowoco-knowledge/hr-intent-service/models/klue-roberta-base-intent-classifier/model.safetensors filter=lfs diff=lfs merge=lfs -text

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

.gitattributes에 klue-roberta의 model.safetensors만 LFS로 등록돼 있고, ax-4.0-light-qlora-classifier/adapter_model.safetensors는 LFS 등록이 안되있는거 같아서 한번 검토해주세요.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

adapter는 깃허브 업로드 제한 용량 안넘어서 그냥 올렸는데 이것도 파일이 크긴하니까 LFS 등록을 하는게 좋을까요?

@BcKmini BcKmini Aug 6, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

같이 등록하면 보기 편할거같아요. 초과가 아니니 안해도 괜찮습니다

@@ -0,0 +1,3 @@
version https://git-lfs.github.com/spec/v1
oid sha256:abf0d7d3bdb89a5ac34abc8cc41a77396bf024f02de6939591b3c555446bd48b
size 442518124

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

오홍.. GitHub LFS 저장소에 있게 되는거죠?

{"id": 1231, "hr_input": "WRK-841 급여명세서 받아서 이번 달 공제 세부 확인 부탁", "intents": [{"intent": "DOCUMENT_REQUEST", "evidence": "급여명세서 받아서"}, {"intent": "PAYROLL_EXPLANATION", "evidence": "이번 달 공제 세부 확인 부탁"}]}
{"id": 1232, "hr_input": "WRK-842: 오늘 새 작업지시서 확인 요청", "intents": [{"intent": "WORK_INSTRUCTION", "evidence": "오늘 새 작업지시서 확인 요청"}]}
{"id": 1233, "hr_input": "신규 WRK-843 계약서 받아서 등록 및 배치 라인 결정 완료", "intents": [{"intent": "OUT_OF_SCOPE", "evidence": null}]}
{"id": 1234, "hr_input": "WRK-844 휴가 종료, 원복 배치 및 급여 정상 지급 안내", "intents": [{"intent": "EMPLOYMENT_CHANGE", "evidence": "휴가 종료"}, {"intent": "WORK_INSTRUCTION", "evidence": "원복 배치"}, {"intent": "PAYROLL_EXPLANA

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

1340개.. 저도 주말에 시간되면 채은님 환경 맞춰서 해보겠습니다

@chaeeunn4
chaeeunn4 merged commit f4f1e82 into main Aug 6, 2026
2 checks passed
@chaeeunn4
chaeeunn4 deleted the docs/add-data-and-models branch August 6, 2026 08:51
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.

3 participants