-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add pre-commit #992
Add pre-commit #992
Conversation
@SunsetWolf will continue to complete this PR |
@SunsetWolf Please add |
Working on some errors prompted by flake8. |
0d496b5
to
f82a447
Compare
scripts/get_data.py
Outdated
@@ -5,5 +5,39 @@ | |||
from qlib.tests.data import GetData | |||
|
|||
|
|||
def get_data( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you have to change this file?
05ddd83
to
c725fd0
Compare
scripts/get_data.py
Outdated
@@ -6,4 +6,4 @@ | |||
|
|||
|
|||
if __name__ == "__main__": | |||
fire.Fire(GetData) | |||
fire.Fire(GetData().qlib_data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure this will work correctly for previous commands?
f876e48
to
60aa277
Compare
Description
Motivation and Context
How Has This Been Tested?
<! --- Put an
x
in all the boxes that apply: --->pytest qlib/tests/test_all_pipeline.py
under upper directory ofqlib
.Screenshots of Test Results (if appropriate):
Types of changes