search

stefan-jansen / machine-learning-for-trading

Code for Machine Learning for Algorithmic Trading, 2nd edition.
https://ml4trading.io
12.87k stars 4.11k forks source link

cp2. edgar_xbrl #250

Closed tonygjwns closed 2 years ago

tonygjwns commented 2 years ago

Describe the bug A brief description of the bug and in which notebook/script it lives.

Bad zip file for 2021 Q1~2022Q2

To Reproduce

Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem. 제목 없음

Environment If you are not using the latest version of the Docker imag:

Additional context Add any other context about the problem here.

stefan-jansen commented 2 years ago

Please check if the download urls have changed. If so, I'll update the example for the next edition.

tonygjwns commented 2 years ago

Please check if the download urls have changed. If so, I'll update the example for the next edition.

The download files had changed from Quarter data to monthly data. I think that made the errors