search

argonne-lcf / THAPI

A tracing infrastructure for heterogeneous computing applications.
Other
22 stars 9 forks source link

Fix tally #269

Closed TApplencourt closed 2 months ago

TApplencourt commented 2 months ago

Sorry for all the other typo, it's clang-format

Focus on _get_uniq_tally

Thanks @nliber

Kerilk commented 2 months ago

Is @colleeneb example summary fixed using this patch?

TApplencourt commented 2 months ago

Yep! 

Explicit memory traffic (BACKEND_OMP) | 16 Hostnames | 192 Processes | 192 Threads |
Explicit memory traffic (BACKEND_ZE) | 16 Hostnames | 384 Processes | 384 Threads |

Which is expected for this app (and 16*24 = 384 so we are good)

colleeneb commented 2 months ago

Is @colleeneb example summary fixed using this patch?

Yep, as @TApplencourt showed, it looks correct!