Skip to content

Conversation

@Derek-Wds
Copy link
Contributor

@Derek-Wds Derek-Wds commented Sep 25, 2020

Hi, please correct me if I put the wrong links for reference. Also, the warning messages looks like this when I tested in on my machine:
Screenshot from 2020-09-25 11-43-44

Thanks!

Derek

@ghost
Copy link

ghost commented Sep 25, 2020

CLA assistant check
All CLA requirements met.

README.md Outdated
Users could create the same dataset with it.
Users could create the same dataset with it.

*Please pay **ATTENTION** that the data is collected from [Yahoo Finance](https://finance.yahoo.com/lookup) and the data might not be perfect. We recommend users to prepare their own data if they have high-quality dataset. For more information, users can refer to the [related document](https://qlib.readthedocs.io/en/latest/component/data.html)*
Copy link
Collaborator

Choose a reason for hiding this comment

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

This link will be more accurate

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, I will fix this.

@you-n-g you-n-g merged commit fc7b968 into microsoft:main Sep 25, 2020
@Derek-Wds Derek-Wds added the documentation Improvements or additions to documentation label Sep 30, 2020
you-n-g pushed a commit to you-n-g/qlib that referenced this pull request Aug 16, 2025
* run the code

* update code

* remove some redundant code

---------

Co-authored-by: xuyang1 <xuyang1@microsoft.com>
gaogen123 pushed a commit to gaogen123/qlib that referenced this pull request Sep 13, 2025
Update data related information
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants