Skip to content

Enhance auto-tune module #1608

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

Merged
merged 25 commits into from
Feb 7, 2024
Merged

Enhance auto-tune module #1608

merged 25 commits into from
Feb 7, 2024

Conversation

yiliu30
Copy link
Contributor

@yiliu30 yiliu30 commented Feb 6, 2024

Type of Change

Enhancement
API changed or not: None

Description

Enhance auto-tune module's internal implementation; no functionality changes.

  • Introduce TuningParam to handle the complex tunable type, such as a list of lists.
  • Separate the Sampler and ConfigSet from ConfigLoader
  • Enhance the BaseConfig's expand
  • discard timeout from TuningConfig

How has this PR been tested?

Pre-CI

Dependency Change?

@yiliu30 yiliu30 self-assigned this Feb 6, 2024
@yiliu30 yiliu30 marked this pull request as ready for review February 6, 2024 04:10
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
@yiliu30 yiliu30 requested a review from yuwenzho February 6, 2024 05:30
@yiliu30 yiliu30 requested a review from xin3he February 6, 2024 09:45
chensuyue and others added 3 commits February 6, 2024 18:03
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
Signed-off-by: yiliu30 <[email protected]>
@yiliu30 yiliu30 merged commit ac47d9b into master Feb 7, 2024
@yiliu30 yiliu30 deleted the ly/enhance_autotune branch February 7, 2024 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants