search

cms-patatrack / pixeltrack-standalone

Standalone Patatrack pixel tracking
Apache License 2.0
17 stars 35 forks source link

Monitor the CPU usage of the event loop #324

Closed fwyzard closed 2 years ago

fwyzard commented 2 years ago

Ideally this should give a result similar to top, but normalised to the number of threads being used. On the other hand, it may give too much importance the impact of the initialisation part.

@makortel what do you think ?

Maybe we should add a way to "warm up" the program, skipping the first N events ?

fwyzard commented 2 years ago

@makortel @markdewing I wasn't able to build with Kokkos; do you want to test the changes before merging this PR, or do you trust it to work the same as for the other backends ?

makortel commented 2 years ago

I'm pretty confident it should work the same, but let me nevertheless test the compilation.

By the way, is sycltest missing on the list on purpose? (or did I miss something)

fwyzard commented 2 years ago

Yes, I skipped it because I didn't have a way to test it at the moment.

makortel commented 2 years ago

kokkos and kokkostest build and work fine with this PR.

I would nevertheless propagate the update to sycltest. I think the update working in all other cases makes it likely to work for sycltest too, and fixing any (possible) compilation errors would be more straightforward than figuring out which infrastructure updates are missing.

fwyzard commented 2 years ago

OK, will do.