-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Remove static pointer holding Pandora #8003
Remove static pointer holding Pandora #8003
Conversation
A new Pull Request was created by @lgray (Lindsey Gray) for CMSSW_6_2_X_SLHC. Remove static pointer holding Pandora It involves the following packages: RecoParticleFlow/PandoraTranslator The following packages do not have a category, yet: RecoParticleFlow/PandoraTranslator @cmsbuild, @nclopezo can you please review it and eventually sign? Thanks. |
What is the plan for making this thread friendly? |
Sorry, I missed that this was SLHC. |
@Dr15Jones I need to speak with John Marshall and Mark Thompson about this. Their framework is inherently not threadsafe. |
merge |
Remove static pointer holding Pandora
An attempt at fixing an issue where rechits provided to pandora would not point back to PFRecHits original provided in memory. Best guess is that putting the rather sneaky circumventions of const-correctness within pandora did not play nicely with being pointed to by a static pointer.