Skip to content

Conversation

@bxdd
Copy link
Collaborator

@bxdd bxdd commented Mar 16, 2021

Description

Support Point-in-time Data Operation

Motivation and Context

How Has This Been Tested?

  • 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:

Types of changes

  • Fix bugs
  • Add new feature
  • Update documentation
@github-actions
Copy link

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

@github-actions github-actions bot added the stale label Nov 10, 2021
@you-n-g you-n-g removed the stale label Nov 10, 2021
report_list = []
while (rs_report.error_code == "0") & rs_report.next():
report_list.append(rs_report.get_row_data())
# 获取一条记录,将记录合并在一起
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use english

requests
pandas
lxml
loguru
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duplicated logurr

```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
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using a more general path will be better

pass


class PExpression(abc.ABC):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MetaClass

@Chaoyingz
Copy link
Contributor

我想在本地把这个pr合并到当前主分支来用,但是有大量的代码冲突。请问这个pr还在开发中吗?没有的话我准备在当前主分支版本的基础上按照这个pr的逻辑进行开发来支持时点数据。

@you-n-g
Copy link
Collaborator

you-n-g commented Mar 2, 2022

@Chaoyingz 还在开发中, 这几天我把这些梳理梳理 整理 一版 出来, 回头咱一起讨论讨论 后面持续优化 :)

@you-n-g you-n-g marked this pull request as draft March 4, 2022 09:36
@you-n-g you-n-g marked this pull request as ready for review March 10, 2022 04:34
@you-n-g
Copy link
Collaborator

you-n-g commented Mar 10, 2022

The development of this PR is finished and it is ready for review now.
Welcome to try this feature.

It will be merged to master soon!

@Chaoyingz

@you-n-g you-n-g merged commit faa99f3 into microsoft:main Mar 10, 2022
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>
you-n-g pushed a commit to you-n-g/qlib that referenced this pull request Aug 16, 2025
* add an error catching in model runner

* check forest & s3e11 & s4e8 & spaceship, change params of forest & s3e26 & spaceship

* fix ci errors

---------

Co-authored-by: TPLin22 <tplin2@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

5 participants