Skip to content

[949] - Remove argument internal in BaseRun::download_data.#953

Merged
you-n-g merged 3 commits intomicrosoft:mainfrom
ChiahungTai:949
Mar 8, 2022
Merged

[949] - Remove argument internal in BaseRun::download_data.#953
you-n-g merged 3 commits intomicrosoft:mainfrom
ChiahungTai:949

Conversation

@ChiahungTai
Copy link
Collaborator

@ChiahungTai ChiahungTai commented Mar 7, 2022

Description

Remove argument internal in BaseRun::download_data.

Motivation and Context

See #949.

How Has This Been Tested?

<! --- Put an x in all the boxes that apply: --->

  • Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
  • If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

  1. Pipeline test:
  2. Your own tests:

Screen Shot 2022-03-07 at 6 32 50 PM

Types of changes

  • Fix bugs
  • Add new feature
  • Update documentation

@you-n-g
Copy link
Collaborator

you-n-g commented Mar 8, 2022

It looks great! Thanks so much!

@you-n-g you-n-g merged commit 57f7ed9 into microsoft:main Mar 8, 2022
@ChiahungTai ChiahungTai deleted the 949 branch March 8, 2022 06:37
@you-n-g you-n-g added the enhancement New feature or request label Mar 8, 2022
qianyun210603 pushed a commit to qianyun210603/qlib that referenced this pull request Mar 23, 2023
…t#953)

* [949] - Remove argument internal in BaseRun::download_data.

* Fix black.

* Fix bug.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants