Skip to content
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

[EP Perf] Fix on EP Perf #20683

Merged
merged 7 commits into from
May 16, 2024
Merged

[EP Perf] Fix on EP Perf #20683

merged 7 commits into from
May 16, 2024

Conversation

yf711
Copy link
Contributor

@yf711 yf711 commented May 14, 2024

Description

  • Partially revert previous change, and
    • Redo concurrency_test_result parser outside of post.py
  • Add support of syncing memtest result to db

Motivation and Context

To fix the error when CI is running on two model groups.

  • When running on two model groups, the previous change wrongly navigates two levels up in the directory after running one model group, while one level is needed. After that, the script can't find another model group.
  • Running on one model group can't repro the issue

@yf711 yf711 marked this pull request as ready for review May 14, 2024 23:28
@yf711 yf711 requested a review from a team as a code owner May 14, 2024 23:28
@jywu-msft jywu-msft requested a review from chilo-ms May 15, 2024 17:29
@yf711 yf711 merged commit 47a178b into main May 16, 2024
97 checks passed
@yf711 yf711 deleted the yifanl/ep_perf_sync_debug branch May 16, 2024 04:38
@jywu-msft jywu-msft added the ep:TensorRT issues related to TensorRT execution provider label May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ep:TensorRT issues related to TensorRT execution provider release:1.18.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants