Skip to content
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

fix: Update runner.py to fix a small bug #282

Merged
merged 2 commits into from
Sep 20, 2024
Merged

Conversation

peteryang1
Copy link
Collaborator

@peteryang1 peteryang1 commented Sep 20, 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--282.org.readthedocs.build/en/282/

@peteryang1 peteryang1 merged commit 8aef3ab into main Sep 20, 2024
7 checks passed
@peteryang1 peteryang1 deleted the peteryang1-patch-1 branch September 20, 2024 06:03
WinstonLiyt pushed a commit that referenced this pull request Sep 20, 2024
* fix: Update runner.py to fix a small bug

* fix CI
WinstonLiyt added a commit that referenced this pull request Sep 20, 2024
* Add runtime measurement for each step and loop in RDLoop.

* refine some codes

* refine the code (#276)

* show variables only when it exists (#277)

* fix: support seed and fix absolute path (#278)

* fix: support seed and fix absolute path

* Absolute path

* lint

* fix: improve_execution_time_in_kaggle_loop (#279)

* improve_execution_time_in_kaggle_loop

* fix CI

* fix CI

* fix CI

* fix: Update runner.py to fix a small bug (#282)

* fix: Update runner.py to fix a small bug

* fix CI

* refine the code

* Update loop.py

* Update rd_loop.py

* Update model_xgb.py

---------

Co-authored-by: XianBW <[email protected]>
Co-authored-by: you-n-g <[email protected]>
Co-authored-by: Xu Yang <[email protected]>
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.

1 participant