Skip to content

fix: partial bug in bench#368

Merged
you-n-g merged 6 commits intomainfrom
bench
Sep 29, 2024
Merged

fix: partial bug in bench#368
you-n-g merged 6 commits intomainfrom
bench

Conversation

@you-n-g
Copy link
Contributor

@you-n-g you-n-g commented Sep 27, 2024

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

📚 Documentation preview 📚: https://RDAgent--368.org.readthedocs.build/en/368/

you-n-g and others added 6 commits September 27, 2024 03:20
* 0 instead of NaN

* remove unused evaluators

* save gen_factor_l_all_rounds

* black reformat

* cache exception
@you-n-g you-n-g marked this pull request as ready for review September 29, 2024 01:59
@you-n-g you-n-g merged commit af9808f into main Sep 29, 2024
@you-n-g you-n-g deleted the bench branch September 29, 2024 01:59
Jensen246 pushed a commit to Jensen246/RD-Agent that referenced this pull request Jun 6, 2025
* Add more log

* Fix eval

* update

* keep ratio <= 1

* feat: cache exception (microsoft#369)

* 0 instead of NaN

* remove unused evaluators

* save gen_factor_l_all_rounds

* black reformat

* cache exception

---------

Co-authored-by: Tim <illking@foxmail.com>
licong01-cloud pushed a commit to licong01-cloud/RD-Agent that referenced this pull request Dec 13, 2025
* Add more log

* Fix eval

* update

* keep ratio <= 1

* feat: cache exception (microsoft#369)

* 0 instead of NaN

* remove unused evaluators

* save gen_factor_l_all_rounds

* black reformat

* cache exception

---------

Co-authored-by: Tim <illking@foxmail.com>
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