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

MSTR-396 : [AI] Refactoring #15

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

MSTR-396 : [AI] Refactoring #15

wants to merge 3 commits into from

Conversation

ekzm8523
Copy link
Member

@ekzm8523 ekzm8523 commented Mar 5, 2023

Issue Number

close: MSTR-396

작업 내역

구현 내용 및 작업 했던 내역

  • dependency 클래스화
  • 형태소 분석기 mecab -> kkma 변경

변경사항

  • 의존성 목록

작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

PR 특이 사항

변경을 하다보니 테스트도 명확하지 않고 코드도 의존성이 꼬여있으며 특히 package 의존성이 가장 꼬여있습니다.
아무래도 기능 정의를 새롭게 하고 새롭게 코드를 만드는게 더 좋을 것 같습니다.
하지만 리뷰가 의미없는건 아니니 더 나은 개선방향이 있다면 남겨주세요 ~!

  • 내용 채점 부분은 코드를 지울지 유지해둘지 고민입니다. 이에 대한 의견 남겨주시면 좋을 것 같네요 :)

ekzm8523 added 3 commits March 5, 2023 17:22
extract application context class
add singleton decorator
change stemmer
@ekzm8523 ekzm8523 requested a review from kshired March 5, 2023 10:42
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.

2 participants