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

Add filter string to mlflow experiment list_recorders function. #512

Merged
merged 5 commits into from
Jul 18, 2021

Conversation

chaosddp
Copy link
Contributor

Description

Current RecorderCollector use default list_recorders function list all ACTIVE_ONLY runs, it may cost much time if there are too much active runs, with mlflow filter string support, we can reduce this number.

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

@you-n-g
Copy link
Collaborator

you-n-g commented Jul 18, 2021

@chaosddp
It looks great now! Thanks :)

@you-n-g you-n-g merged commit 3183a23 into microsoft:main Jul 18, 2021
@chaosddp chaosddp deleted the feature.mlflow_filter_str branch July 26, 2021 06:05
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