Skip to content

Conversation

@kimjoe1850
Copy link

늦어서 죄송합니다..

paulben0410
paulben0410 previously approved these changes Jan 3, 2023
00HS
00HS previously approved these changes Jan 20, 2023
yurishin929
yurishin929 previously approved these changes Feb 8, 2023
Copy link

@hjm507 hjm507 left a comment

Choose a reason for hiding this comment

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

너무 늦어서 죄송합니다 ㅜㅜ

여러 형태소 분석기를 한 눈에 볼 수 있어서 좋은 것 같네요.

다만 블로그 업로드 시 어떻게 될 지는 모르겠는데 이미지 부분 가독성이 조금 떨어질 것 같아서
이미지 크기를 수정하거나 각 항목마다 가로로 한번에 나열해서 비교해서 볼 수 있게 하면 좋을 것 같아요!

@beautifulchoi
Copy link

늦어서 죄송합니다,, 리뷰를 한 줄 알았는데 12월 리뷰를 건너 뛰었었네요 ㅠㅠ

한국어 자연어 처리는 저에게는 접해보지 않은 분야라서 낯설었는데 한국어 형태소 분석기(아마 토크나이저? 인 것 같은데 맞는지 모르겠습니다...,,)를 통한 데이터 전처리에 대해 조금은 알 수 있었던 것 같습니다.

저도 진민님과 같은 의견으로 노트북 출력결과가 제각기라서 가독성이 좀 떨어지는 것 같고, 혹시 후속 글을 쓰신다면 각 형태소 분석기를 비교하여 모델을 테스트 해보면 좋을 것 같습니다!


![graph](../assets/images/post-POS-tagger/graph.png)

그래프만 놓고 봤을 땐 Mecab이 가장 빠르게 나온다.

Choose a reason for hiding this comment

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

분석기간 시간 차이가 어마어마 한 차이까지는 아니여도 꽤 많이 나는데 왜 그런지 더 알 수 있으면 좋을 것 같습니다!

@hoqn hoqn added ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 and removed 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ labels Feb 10, 2023
이미지 수정 및 일부 내용 추가
@kimjoe1850 kimjoe1850 dismissed stale reviews from yurishin929, 00HS, and paulben0410 via 1ebe463 April 7, 2023 05:25
@SSUHYUNKIM SSUHYUNKIM added 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ and removed ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 labels Apr 7, 2023
@hoqn hoqn removed 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ labels Jul 11, 2023
@hoqn hoqn added the 💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때 label Jul 11, 2023
@hoqn hoqn merged commit 0e27c7d into GDSC-University-of-Seoul:master Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants