Skip to content

fix(serve): dedicated INFERENCE_RECOMMENDER telemetry feature + type workload param#6028

Merged
zhaoqizqwang merged 1 commit into
aws:masterfrom
ZealSV:fix/ai-inference-recommender-review-nits
Jul 20, 2026
Merged

fix(serve): dedicated INFERENCE_RECOMMENDER telemetry feature + type workload param#6028
zhaoqizqwang merged 1 commit into
aws:masterfrom
ZealSV:fix/ai-inference-recommender-review-nits

Conversation

@ZealSV

@ZealSV ZealSV commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Address review nits on the AI inference recommender:

  • Add Feature.INFERENCE_RECOMMENDER telemetry enum + FEATURE_TO_CODE mapping; tag generate_deployment_recommendations, start_benchmark, and show_result with it instead of MODEL_CUSTOMIZATION.
  • Type generate_deployment_recommendations(workload=...) as Optional[Union[Workload, str]] to match the other params.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

…workload param

Address review nits on the AI inference recommender:
- Add Feature.INFERENCE_RECOMMENDER telemetry enum + FEATURE_TO_CODE mapping;
  tag generate_deployment_recommendations, start_benchmark, and show_result
  with it instead of MODEL_CUSTOMIZATION.
- Type generate_deployment_recommendations(workload=...) as
  Optional[Union[Workload, str]] to match the other params.
@ZealSV
ZealSV requested a deployment to manual-approval July 15, 2026 18:19 — with GitHub Actions Waiting
@ZealSV
ZealSV requested a deployment to manual-approval July 15, 2026 18:19 — with GitHub Actions Waiting
@zhaoqizqwang
zhaoqizqwang merged commit 4b3c156 into aws:master Jul 20, 2026
3 of 5 checks passed
@github-actions github-actions Bot mentioned this pull request Jul 24, 2026
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