Skip to content

AIESW-41600 Consolidate throughput and latency tests for npu3 - #9980

Open
rchane wants to merge 5 commits into
Xilinx:masterfrom
rchane:AIESW-41600
Open

AIESW-41600 Consolidate throughput and latency tests for npu3#9980
rchane wants to merge 5 commits into
Xilinx:masterfrom
rchane:AIESW-41600

Conversation

@rchane

@rchane rchane commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Problem solved by the commit

https://jira.xilinx.com/browse/AIESW-41600

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

Consolidating throughput and latency xrt-smi validate tests for npu3, skip TCT tests as well

How problem was solved, alternative solutions (if any) and why they were rejected

Dropped runlist-latency and runlist-throughput (return N/A on npu3, shows up as "skipped" in output) and moved logic of runlist-throughput to default throughput test, skipped TCT tests on npu3

Risks (if any) associated the changes in the commit

Test deprecation, can break pipelines if numbers are expected when parsing output for the tests

What has been tested and how, request additional testing if necessary

Testing on STX/npu3

Documentation impact (if any)

Will need to update xrt-smi user guide

Signed-off-by: Ryan Chane <rchane@xilinx.com>
@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@aktondak

Copy link
Copy Markdown
Collaborator

Looks good to me.
Can you post the result with latest VTD and try to merge this soon ?

Signed-off-by: Ryan Chane <rchane@xilinx.com>
@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Signed-off-by: Ryan Chane <rchane@xilinx.com>
@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Signed-off-by: Ryan Chane <rchane@xilinx.com>
@rchane
rchane marked this pull request as ready for review August 26, 2026 17:45
@rchane
rchane requested a review from aktondak August 26, 2026 17:46
@rchane rchane added the do not merge hold off on merging label Aug 26, 2026
@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge hold off on merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants