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

Komoran 객체 생성 시 모델 지정이 필요하도록 변경 #19

Merged
merged 3 commits into from
Apr 11, 2019

Conversation

9bow
Copy link
Member

@9bow 9bow commented Apr 10, 2019

#18 에 따라 Komoran 객체 생성 시 반드시 모델을 지정하도록 변경하였습니다.
(기존에는 Full 모델이 기본이었습니다.)

이는 KOMORAN에서도 동일하게 존재하였던 문제로,
사용자 사전 등의 적용 시 문제가 발생하는 것을 막기 위해 객체 생성 시 기본 모델을 지정하지 않습니다.

@9bow 9bow added the bug Something isn't working label Apr 10, 2019
@9bow 9bow added this to the 0.1.3 milestone Apr 10, 2019
@9bow 9bow requested review from shin285 and gamamoe April 10, 2019 13:09
@9bow 9bow self-assigned this Apr 10, 2019
@9bow
Copy link
Member Author

9bow commented Apr 10, 2019

이번 Pull Request까지의 적용을 기준으로 0.1.3으로 버전을 업그레이드합니다.

@shin285 @dolpang2
0.1.3 릴리즈를 위한 리뷰 요청드립니다.

@@ -18,7 +18,7 @@
:linenos:

from PyKomoran import *
komoran = Komoran()
komoran = Komoran(DEFAULT_MODEL['FULL'])
Copy link
Member

Choose a reason for hiding this comment

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

튜토리얼에서는 LIGHT 모델로 사용자들을 유도하는 게 더 좋지 않을까 생각합니다. 정량적인 수치는 모르겠지만, 일반적인 사용용도에서는 LIGHT 모델이 체감상 더 나았던 것 같아요.

Copy link
Member

Choose a reason for hiding this comment

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

@shin285 어떻게 생각하세요?

Copy link
Member

Choose a reason for hiding this comment

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

저도 리뷰할때 고민했습니다. 근호님도 고민을 하셨다면 저도 LIGHT가 나을 것 같습니다.

Copy link
Member

Choose a reason for hiding this comment

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

사실 LIGHT, FULL 네이밍 자체가 조금 FULL 이 좋은거 아니야? 라고 혹하게 되는 느낌이라.. 이것도 한번 아젠다에 추가하는게 좋을 것 같네요.

Copy link
Member

Choose a reason for hiding this comment

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

네 회의 아젠다에 추가하겠습니다.

Copy link
Member

Choose a reason for hiding this comment

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

@dolpang2 LIGHT로 변경 시 예제 문장에 오분석 케이스가 발생하여 일단 FULL 버전으로 가기로 하였습니다. 참고부탁드립니다.

Copy link
Member

@gamamoe gamamoe left a comment

Choose a reason for hiding this comment

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

튜토리얼에서의 사소한 의견만 추가하였습니다. 우선 Approve 하겠습니다

@9bow
Copy link
Member Author

9bow commented Apr 11, 2019

기존 예제 문장 이슈로 우선은 FULL로 진행하고, 추후 tutorials쪽에서 LIGHT를 반영하도록 하겠습니다.

@9bow 9bow merged commit 16b0b76 into master Apr 11, 2019
@9bow 9bow deleted the #18_need_model_when_init_Komoran branch April 11, 2019 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants