ikegami-yukino / oseti

Dictionary based Sentiment Analysis for Japanese
MIT License
91 stars 15 forks source link

Update mecab to mecab-python3 #6

Closed xia0nan closed 1 year ago

xia0nan commented 2 years ago

Need to update mecab to mecab-python3 or installation will throw errors.

ikegami-yukino commented 1 year ago

@xia0nan What is this Pull-request?

xia0nan commented 1 year ago

@ikegami-yukino in the requirements.txt, one dependency is listed: mecab but it will cause installation failure. it works if you change it to mecab-python3 (https://pypi.org/project/mecab-python3/)

ikegami-yukino commented 1 year ago

@xia0nan What is an error message? I will maintain the mecab module.

xia0nan commented 1 year ago

@ikegami-yukino you can try in google colab with the !pip install oseti it will throw you error

Collecting mecab
  Downloading mecab-0.996.3.tar.gz (62 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 62.6/62.6 KB 4.7 MB/s eta 0:00:00
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> See above for output.
ikegami-yukino commented 1 year ago

@xia0nan This problem is that MeCab C++ library is not installed.

Please try bellow.

!apt install aptitude
!aptitude install mecab libmecab-dev mecab-ipadic-utf8 git make curl xz-utils file -y
ikegami-yukino commented 1 year ago

No replies, so I'm closing this PR...