Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Readme 업데이트에 관하여 #25

Open
EugeneYoo opened this issue Oct 12, 2021 · 1 comment
Open

Readme 업데이트에 관하여 #25

EugeneYoo opened this issue Oct 12, 2021 · 1 comment

Comments

@EugeneYoo
Copy link

EugeneYoo commented Oct 12, 2021

우선 올려주신 코드 덕분에 뉴스데이터 확보를 편하게 할 수 있었습니다. 감사합니다.

하나 건의를 드리고자 issue 올립니다.

지금은 Readme에 종료 관련된 내용이 따로 없는 것으로 알고 있습니다.

제가 사용해본 결과 오늘 날짜를 포함하여 크롤링을 진행하는 경우 종료가 되지 않고 계속 크롤링을 하는 것 같습니다.

예를 들어 기간을 (2021, 9, 2021, 10)으로 잡는 경우 오늘 기준(2021.10.12)으로 10월 달이 끝나지 않았기 때문에

계속 실행이 되며 csv를 업데이트 하는 것으로 확인했습니다.

계속 크롤링을 진행해야 하는 경우라면 유용하겠지만, 처음에는 이걸 보고 중간에 종료를 해도 되는지 의문이 생겼었습니다.

저같은 경우 저장되는 csv 파일을 열어서 오늘 날짜로 어느 정도 저장된 걸 확인하고 크롤러를 종료했습니다.

제가 확인한 것이 맞다면 현재 날짜가 포함되는 크롤링에 대한 설명을 추가해주셨으면 합니다.

'임의로 종료하셔도 됩니다' 라던지, 아니면 아예 날짜까지 지정하게 업데이트를 해주시면 더 좋은 소스 코드가 될 것 같습니다.

추가로 output이 바탕 화면에 저장된다는 내용을 적어주셨으면 합니다. 해당 부분을 몰라서 파일을 찾는 분들이 있을 것 같습니다.

감사합니다.

@EugeneYoo EugeneYoo changed the title 종료 관련 Readme 업데이트에 관하여 Readme 업데이트에 관하여 Oct 12, 2021
@lumyjuwon
Copy link
Owner

해당 내용 관련해서 PR 해주시면 반영 검토 해보겠습니다 ~

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

No branches or pull requests

2 participants