Skip to content

Add BaseCollector #324

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

Merged
merged 1 commit into from
Mar 12, 2021
Merged

Add BaseCollector #324

merged 1 commit into from
Mar 12, 2021

Conversation

zhupr
Copy link
Collaborator

@zhupr zhupr commented Mar 10, 2021

Description

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

@zhupr zhupr force-pushed the add_base_collector branch from cf5bee3 to 6f18778 Compare March 10, 2021 10:59
self.save_stock(symbol, df)
return _result

def save_stock(self, symbol, df: pd.DataFrame):
Copy link
Collaborator

Choose a reason for hiding this comment

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

rename stock to instrument will make it sound more general


def collector_data(self):
"""collector data"""
logger.info("start collector yahoo data......")
Copy link
Collaborator

Choose a reason for hiding this comment

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

yahoo ?

@zhupr zhupr force-pushed the add_base_collector branch from 6f18778 to cef1c30 Compare March 12, 2021 02:22
@zhupr zhupr force-pushed the add_base_collector branch from cef1c30 to 42be8ac Compare March 12, 2021 02:30
@you-n-g you-n-g merged commit df56e3b into microsoft:main Mar 12, 2021
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.

2 participants