search

trackreco / mkFit

Vectorized, Parallelized Tracking
https://trackreco.github.io/
Apache License 2.0
17 stars 15 forks source link

Builder release memory per event #383

Closed osschar closed 2 years ago

osschar commented 2 years ago

Memory allocated for TrackCand vectors is released on every event / iteration -- before it was growing to accommodate the largest event seen so far.

Further, overlap hit-storage vector that was member of CombinedCandidate has been moved as member variable in TrackCand -- thus it does not need to be allocated as a vector per seed anymore.

igProf MEM_LIVE 50 ttbar pu 50 events: http://xrd-cache-1.t2.ucsd.edu/matevz/igProf/igprof-navigator.py/IgProf.50

DQM file for 7 iterations, ttbar 50 isin phi3:/data2/matevz/CMSSW_11_2_0/src/mem_test/ ... I don't have relevant reference files on hand.

osschar commented 2 years ago

@makortel can you please check?

makortel commented 2 years ago

As far as I can tell (including the IgProf report), looks good!

slava77 commented 2 years ago

As far as I can tell (including the IgProf report), looks good!

I merged based on this