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

[GPU] Add a new option for DumpProfilingData #23006

Merged

Conversation

andrew-k-park
Copy link
Contributor

Details:

  • Enable collecting profiling data only at iterations with requested range
  • For example for dump profiling data only when iteration is from 10 to 20, you can use OV_GPU_DumpProfilingDataIteration='10..20'

Tickets:

  • 133260

@andrew-k-park andrew-k-park added the category: GPU OpenVINO GPU plugin label Feb 22, 2024
@andrew-k-park andrew-k-park requested review from a team as code owners February 22, 2024 07:07
@andrew-k-park andrew-k-park force-pushed the dump_profiling_data_inter branch 3 times, most recently from 89db6fa to 76522a6 Compare February 22, 2024 11:08
@yeonbok
Copy link
Contributor

yeonbok commented Feb 24, 2024

Also, there is an unexpected behavior on network->get_current_iteration_num which returns cur_iter + 1 value. i.e., The current iteration presented in OV_GPU_Verbose starts from 0, but the iteration dumped by DumpProfilingData starts from 1. Could you fix that in this PR too?

@andrew-k-park andrew-k-park force-pushed the dump_profiling_data_inter branch 2 times, most recently from a0dbfe6 to bcff601 Compare February 26, 2024 06:58
@andrew-k-park
Copy link
Contributor Author

Also, there is an unexpected behavior on network->get_current_iteration_num which returns cur_iter + 1 value. i.e., The current iteration presented in OV_GPU_Verbose starts from 0, but the iteration dumped by DumpProfilingData starts from 1. Could you fix that in this PR too?

Corrected so that dumped iteration and the actual network's iteration were the same.

@andrew-k-park andrew-k-park force-pushed the dump_profiling_data_inter branch from bcff601 to 7dd2d5e Compare February 26, 2024 10:45
@andrew-k-park andrew-k-park force-pushed the dump_profiling_data_inter branch from 777df62 to 67e8c72 Compare February 26, 2024 23:54
@yeonbok yeonbok enabled auto-merge February 26, 2024 23:58
@yeonbok yeonbok added this pull request to the merge queue Feb 27, 2024
Merged via the queue into openvinotoolkit:master with commit ddf7c44 Feb 27, 2024
90 checks passed
@yeonbok yeonbok deleted the dump_profiling_data_inter branch February 27, 2024 04:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: GPU OpenVINO GPU plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants