Support Point-in-time Data Operation #343
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This PR is stale because it has been open for a year with no activity. Remove the stale label or comment on the PR otherwise this will be closed in 5 days |
you-n-g
reviewed
Dec 9, 2021
| report_list = [] | ||
| while (rs_report.error_code == "0") & rs_report.next(): | ||
| report_list.append(rs_report.get_row_data()) | ||
| # 获取一条记录，将记录合并在一起 |
| requests | ||
| pandas | ||
| lxml | ||
| loguru |
scripts/data_collector/pit/README.md
Outdated
| ```bash | ||
|
|
||
| # download from baostock.com | ||
| python collector.py download_data --source_dir /data1/v-xiabi/qlib/pit/csv_2 --start 2000-01-01 --end 2020-01-01 --interval quarterly |
There was a problem hiding this comment.
Using a more general path will be better
qlib/data/base.py
Outdated
| @@ -224,3 +229,362 @@ class ExpressionOps(Expression): | |||
| """ | |||
|
|
|||
| pass | |||
|
|
|||
|
|
|||
| class PExpression(abc.ABC): | |||
|
我想在本地把这个pr合并到当前主分支来用，但是有大量的代码冲突。请问这个pr还在开发中吗？没有的话我准备在当前主分支版本的基础上按照这个pr的逻辑进行开发来支持时点数据。 |
|
@Chaoyingz 还在开发中， 这几天我把这些梳理梳理 整理 一版 出来， 回头咱一起讨论讨论 后面持续优化 :) |
|
The development of this PR is finished and it is ready for review now. It will be merged to master soon! |
qianyun210603
pushed a commit
to qianyun210603/qlib
that referenced
this pull request
Mar 23, 2023
* add period ops class * black format * add pit data read * fix bug in period ops * update ops runnable * update PIT test example * black format * update PIT test * update tets_PIT * update code format * add check_feature_exist * black format * optimize the PIT Algorithm * fix bug * update example * update test_PIT name * add pit collector * black format * fix bugs * fix try * fix bug & add dump_pit.py * Successfully run and understand PIT * Add some docs and remove a bug * mv crypto collector * black format * Run succesfully after merging master * Pass test and fix code * remove useless PIT code * fix PYlint * Rename Co-authored-by: Young <afe.young@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Support Point-in-time Data Operation
Motivation and Context
How Has This Been Tested?
pytest qlib/tests/test_all_pipeline.pyunder upper directory ofqlib.Screenshots of Test Results (if appropriate):
Types of changes