Skip to content
Snippets Groups Projects

Make sure there is a TGeoNavigator per thread before calling Contains

Merged Izaac Sanderswood requested to merge isanders-fix-mt-nav into master

Required to make FTHitEfficiencyMonitor work when multi-threaded, as currently the geometry navigation is not configured for multi-threading.

Requires LHCb!4652 (merged) where the number of threads is set.

I provided a detailed description of the relevant functions in LHCb!4652 (merged), and then on slide 8 (titled "Usage") of this very old presentation describing the parallelisation of the ROOT Geometry library (the only relevant documentation I could find) it shows the required change in this MR https://indico.cern.ch/event/193979/contributions/357987/attachments/280434/392148/ParallelizingROOTgeometry.pdf

I think this is the correct ROOT way to do it, but in future we can probably find a better way to do it.


Validated by

  • Core Software
  • RTA
  • Simulation
Edited by Gloria Corti

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading