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
Temporary fix to clear TrackerHitAssociator maps each event #8990
Temporary fix to clear TrackerHitAssociator maps each event #8990
Conversation
please test |
The tests are being triggered in jenkins. |
A new Pull Request was created by @wmtan for CMSSW_7_5_X. Temporary fix to clear TrackerHitAssociator maps each event It involves the following packages: SimTracker/TrackerHitAssociation @civanch, @nclopezo, @danduggan, @mdhildreth, @cmsbuild, @deguio can you please review it and eventually sign? Thanks. |
Temporary fix to clear TrackerHitAssociator maps each event
The change to TrackerHitAssociator to support the consumes interface was done in a suboptimal way that, among other things, did not release the PSimHit maps until the beginning of the next event, causing memory issues. This PR is a simple temporary fix to clear the maps at the proper time.
This fix was requested by David Lange.
Since my tests of this PR are still in progress, it would be prudent to test this PR before merging.
NOTE: The consumes interface for TrackerHitAssociator will later be re-implemented in a more robust and less error-prone way very soon.