ko-nlp / Korpora

Korean corpus repository
Creative Commons Attribution 4.0 International
694 stars 80 forks source link

utils.fetch 를 positional arguments 로 이용하는 함수들의 수정이 필요합니다. #57

Closed lovit closed 4 years ago

lovit commented 4 years ago

최근에 fetch 함수에 method argument 가 arguments 중간에 새로 들어오면서 이전에 fetch 함수를 positional arguments 로 이용하던 함수들에서 문제가 발생했습니다.

(now) in korpora_nsmc.py

fetch(info['url'], local_path, 'nsmc', force_download)

위의 상황에서 fetch 함수에 들어가는 인자들을 확인하기 위하여 모든 인자를 풀어서 입력하던제

(desired)

fetch(info['url'], local_path, 'nsmc', info['method'], force_download)

혹은 keyword arguments 로 이용하는 것이 안전해 보입니다.

(desired)

fetch(
  url=info['url'],
  local_path=local_path,
  ..
)
lovit commented 4 years ago

이 이슈는 namuwki#4 branch 에서 해결하겠습니다. 이후에 코드 통일 이슈에서 이 이슈를 다시 다뤄야 합니다.