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

Local filesystem fixes #25

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

primerano
Copy link

I updated the LocalClient to prefix the files with the DOWNLOAD_PATH. I'm not sure if this breaks the intended design of the LocalClient but I didn't expect the downloaded files to be in my source code directory.

I have

CLIENT_TYPE=LOCAL
DOWNLOAD_PATH=/a_filesystem_with_a_lot_of_space

With this change the files pulled from edgar end up in in

  • /a_filesystem_with_a_lot_of_space/documents
  • /a_filesystem_with_a_lot_of_space/edgar

Let me know if this looks ok or if there is a different approach I should take.

Thanks.

Side note. the full.txt changes are what I needed to get this running on Python 3.6. What is the suggested Python version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant