Skip to content

Update readme and setup.py#258

Merged
you-n-g merged 1 commit intomicrosoft:mainfrom
Derek-Wds:bug
Feb 4, 2021
Merged

Update readme and setup.py#258
you-n-g merged 1 commit intomicrosoft:mainfrom
Derek-Wds:bug

Conversation

@Derek-Wds
Copy link
Contributor

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

@Derek-Wds Derek-Wds added the bug Something isn't working label Feb 4, 2021
@Derek-Wds Derek-Wds requested a review from you-n-g February 4, 2021 04:15
Copy link
Contributor

@pingsutw pingsutw left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks @Derek-Wds

@you-n-g you-n-g merged commit 06158fb into microsoft:main Feb 4, 2021
you-n-g added a commit to you-n-g/qlib that referenced this pull request Aug 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants