Skip to content

refactor: PMI 함수를 soynlp.word에서 soynlp.utils로 이동 (#267) - #273

Merged
lovit merged 1 commit into
refactor-2026from
feature/267
Mar 10, 2026
Merged

refactor: PMI 함수를 soynlp.word에서 soynlp.utils로 이동 (#267)#273
lovit merged 1 commit into
refactor-2026from
feature/267

Conversation

@lovit

@lovit lovit commented Mar 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • soynlp/utils/pmi.py 신규 생성: _as_diag, _logarithm_and_ppmi, pmi 함수 이동
  • soynlp/utils/__init__.py: pmi를 public export에 추가
  • soynlp/word/pmi.py: soynlp.utils.pmi re-export + DeprecationWarning 추가 (하위 호환 유지)
  • tests/unit/test_pmi.py: 새 import 경로 기준으로 변경, deprecated 경로 경고 검증 테스트 추가

Closes #267

Test plan

  • uv run pytest tests/unit/test_pmi.py — 3 passed
  • uv run pre-commit run --all-files — all passed

🤖 Generated with Claude Code

- soynlp/utils/pmi.py: PMI 관련 함수(_as_diag, _logarithm_and_ppmi, pmi) 이동
- soynlp/utils/__init__.py: pmi를 public export에 추가
- soynlp/word/pmi.py: soynlp.utils.pmi를 deprecated re-export로 교체
  (DeprecationWarning 포함, 하위 호환 유지)
- tests/unit/test_pmi.py: soynlp.utils.pmi 기준으로 import 변경,
  deprecated 경로 경고 검증 테스트 추가

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lovit
lovit merged commit 3110b52 into refactor-2026 Mar 10, 2026
2 checks passed
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.

1 participant