Skip to content

test: Add config parameter "execution_context_allocation_strategy" to TensorRT backend#8150

Merged
yinggeh merged 9 commits into
mainfrom
yinggeh-DLIS-8281-allocation-strategy-parameter
Apr 18, 2025
Merged

test: Add config parameter "execution_context_allocation_strategy" to TensorRT backend#8150
yinggeh merged 9 commits into
mainfrom
yinggeh-DLIS-8281-allocation-strategy-parameter

Conversation

@yinggeh
Copy link
Copy Markdown
Contributor

@yinggeh yinggeh commented Apr 16, 2025

What does the PR do?

Add unittests to validate server behavior with various allocation_strategy parameter values.

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

Check the conventional commit type
box here and add the label to the github PR.

  • test

Related PRs:

Where should the reviewer start?

Test plan:

All TensorRT related tests
L0_model_config--base

  • CI Pipeline ID:
    27050513

Caveats:

Background

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

@yinggeh yinggeh added the PR: test Adding missing tests or correcting existing test label Apr 16, 2025
@yinggeh yinggeh self-assigned this Apr 16, 2025
@yinggeh yinggeh requested a review from rmccorm4 April 16, 2025 18:00
@rmccorm4 rmccorm4 requested a review from tanmayv25 April 16, 2025 20:43
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Related thread on this testing: triton-inference-server/tensorrt_backend#109 (comment)

Feature PR corresponding to these tests: triton-inference-server/tensorrt_backend#108

Comment thread qa/L0_model_config/test.sh Outdated
@yinggeh yinggeh requested a review from rmccorm4 April 18, 2025 22:42
Comment thread qa/L0_sequence_batcher/test.sh
@yinggeh yinggeh requested a review from rmccorm4 April 18, 2025 23:00
@yinggeh yinggeh changed the title test: Parameterize TensorRT allocation strategy test: Add config parameter "execution_context_allocation_strategy" to TensorRT backend Apr 18, 2025
@yinggeh yinggeh merged commit 901b0e9 into main Apr 18, 2025
3 checks passed
@yinggeh yinggeh deleted the yinggeh-DLIS-8281-allocation-strategy-parameter branch April 18, 2025 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: test Adding missing tests or correcting existing test

Development

Successfully merging this pull request may close these issues.

2 participants