search

lovit / KR-WordRank

비지도학습 방법으로 한국어 텍스트에서 단어/키워드를 자동으로 추출하는 라이브러리입니다
Other
354 stars 57 forks source link

Update README.md #11

Closed br-kim closed 4 years ago

br-kim commented 4 years ago

안녕하세요. 좋은 패키지 만드신 거 같아서 한번 사용해보려고 하는데 scipy 패키지가 필요한거 같은데 setup.py에 설정되어 있지 않습니다. 패키지 버전은 어떻게 설정해야 될지 모르겠어서 readme만 수정했습니다. 풀리퀘스트 보내본적이 없어서.. 미흡하더라도 양해부탁드립니다.

lovit commented 4 years ago

안녕하세요 @br-kim 님, 도움 감사합니다. 저 역시 이 패키지를 만들던 시절, 아는 것들이 적어 dependency 관리를 하지 않았었네요. 참고로 말씀드리면 이는 setup.py 파일의 install_requires 에서 설정하면 dependency 가 걸려 scipy 를 함께 설치합니다.

Pull request 를 보내고 싶으실지도 모른 다는 생각이 들어 install requires 부분은 당분간 수정하지 않겠습니다. setup.py 를 수정하셔서 PR 보내주신다면, 리뷰 후 머징하겠습니다.

아니라면, 보내주신 readme 만 merge 하고, 이후 setup.py 는 수정해두도록 하겠습니다. 커멘트와 PR 감사합니다.

br-kim commented 4 years ago

아.. 네 감사합니다 그런데 제가 잘 모르는 부분이 scipy 버전이 몇버전 이상부터 작동하는지 모르겠어서요. 버전없이 하면 구버전으로 되는데도 괜히 업데이트 해야되고 그럴까봐 섣불리 작성하기가 좀 그렇네요. 관련 패키지를 많이 써보지도 않아서.. 그냥 편하신대로 하시면 될 것 같습니다. 좋은 패키지 만들어주셔서 감사하고 따로 신경 써 주신것도 감사합니다.

br-kim commented 4 years ago

아.. 그리고 또 sklearn 패키지도 포함해주셔야 할것 같네요.

lovit commented 4 years ago

아.. 네 감사합니다 그런데 제가 잘 모르는 부분이 scipy 버전이 몇버전 이상부터 작동하는지 모르겠어서요. 버전없이 하면 구버전으로 되는데도 괜히 업데이트 해야되고 그럴까봐 섣불리 작성하기가 좀 그렇네요. 관련 패키지를 많이 써보지도 않아서.. 그냥 편하신대로 하시면 될 것 같습니다. 좋은 패키지 만들어주셔서 감사하고 따로 신경 써 주신것도 감사합니다.

아.. 그리고 또 sklearn 패키지도 포함해주셔야 할것 같네요.

넵 커멘트 남겨주신 부분들 신경써서 수정해 두도록 하겠습니다. 감사합니다!

lovit commented 4 years ago

12 의 이슈를 commit 8695281 으로 해결하였습니다.