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

ENH: --force-redownload optional parameter. #1973

Closed
wants to merge 1 commit into from
Closed

ENH: --force-redownload optional parameter. #1973

wants to merge 1 commit into from

Conversation

JoaoAparicio
Copy link
Contributor

zipline run calls function load_market_data. This function has a
cooldown of 1 hour. If it downloaded data less than 1 hour ago it won't
download again.

This has an inconvenient side-effect: if zipline run is executed,
data is not cached, data is downloaded but for some reason data is
still not present after download (like what happens with issue #1965),
and if then the issue is fixed, zipline run can still not be executed until
the cooldown time has expired. This can then lead users to open
new issues which seem related to the original one, but are not (see
for example issue #1957 and discussion therein).

This commit adds an optional --force-redownload flag to zipline run.
The default is set to false.

zipline run calls function load_market_data. This function has a
cooldown of 1 hour. If it downloaded data less than 1 hour ago it won't
download again.

This has an inconvenient side-effect: if zipline run is executed,
data is not cached, data is downloaded but for some reason data is
still not present after download (like what happens with issue #1965),
if then the issue is fixed, zipline run can still not be executed until
the cooldown time has expired. This can then lead users to open
new issues which seem related to the original one, but are not (see
for example issue #1957 and discussion therein).

This commit adds an optional --force-redownload flag to zipline run.
The default is set to false.
@coveralls
Copy link

coveralls commented Oct 4, 2017

Coverage Status

Coverage decreased (-0.01%) to 87.197% when pulling 8d1bafa on JoaoAparicio:forceredownload into 5421ec0 on quantopian:master.

@JoaoAparicio JoaoAparicio changed the title ENH: --force-reload optional parameter. ENH: --force-redownload optional parameter. Oct 8, 2017
@JoaoAparicio JoaoAparicio closed this by deleting the head repository Apr 10, 2023
@andycwang
Copy link

andycwang commented Apr 10, 2023 via email

@vangus11
Copy link

@.***邮箱联系我,谢谢!

全部屏蔽了

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.

4 participants