Fix $volume normalization issue #792
Conversation
|
@2young-2simple-sometimes-naive Thanks for your contribution! |
black formatting
black formatting
black formatting
|
Besides, will your results change significantly if you add this patch? |
|
The format errors are listed in the CI output. |
Depends on the selection of stocks. My strategy has a lot of low volumes stocks, and the fix does not change the IC/RIC significantly. It turns out the volume info is not that important in my stock pool. |
|
It looks great to me. |
* Fix $volume normalization issue Fix: microsoft#765 * black formatting black formatting * black formatting black formatting * black formatting black formatting
…t#792) * refactor: use dynamic input path and update template loader * fix: update include syntax for data source in prompts.yaml * add customization path * docs: update prompts for ensemble scoring and metric direction * chore: remove obsolete data_science/share.yaml file
2 participants
Fix: #765
Description
change $volume to $volume+1e-12 such that low volume stocks will not create infinity in features.
Motivation and Context
https://github.com//issues/765 $volume creates infinity for low volume stocks.How Has This Been Tested?
pytest qlib/tests/test_all_pipeline.pyunder upper directory ofqlib.Screenshots of Test Results (if appropriate):
Types of changes