Skip to content

test: Test safe GetElementCount and GetByteSize APIs at common and core repos#8689

Merged
yinggeh merged 2 commits intomainfrom
yinggeh/tri-737-psirt-triton-inference-servercore-integer-overflow
Mar 12, 2026
Merged

test: Test safe GetElementCount and GetByteSize APIs at common and core repos#8689
yinggeh merged 2 commits intomainfrom
yinggeh/tri-737-psirt-triton-inference-servercore-integer-overflow

Conversation

@yinggeh
Copy link
Copy Markdown
Contributor

@yinggeh yinggeh commented Mar 10, 2026

What does the PR do?

  • Test new GetElementCount and GetByteSize added to common and core repos.
  • Reuse backend_tensor_size_test.cc and rename to tensor_size_test.cc.

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:

triton-inference-server/core#475
triton-inference-server/common#150

Where should the reviewer start?

Test plan:

  • CI Pipeline ID:
    45921643

Caveats:

Background

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

  • closes GitHub issue: #xxx

@yinggeh yinggeh requested review from pskiran1 and whoisj March 10, 2026 03:37
@yinggeh yinggeh self-assigned this Mar 10, 2026
@yinggeh yinggeh added the PR: test Adding missing tests or correcting existing test label Mar 10, 2026
@yinggeh yinggeh merged commit 06afae5 into main Mar 12, 2026
3 checks passed
@yinggeh yinggeh deleted the yinggeh/tri-737-psirt-triton-inference-servercore-integer-overflow branch March 12, 2026 04:40
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