-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Made GeometricDet thread-safe #2799
Made GeometricDet thread-safe #2799
Conversation
GeometricDet::setGeographicalID is now non-const. To allow children of a GeometricDet to have their geomgraphicalID set we added a component(size_t) method which gives non-const access to a child. All classes which were modifying the GeometricDet have been updated to support the new interface.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_1_X. Made GeometricDet thread-safe It involves the following packages: Geometry/TrackerNumberingBuilder @civanch, @Dr15Jones, @ianna, @mdhildreth, @cmsbuild, @nclopezo, @Degano, @ktf can you please review it and eventually sign? Thanks. |
Hi Chris, I don't see problems on the code side, at least at a quick glance. |
Thanks for taking a look. |
+1 |
Geometry/TrackerNumberingBuilder -- Made GeometricDet thread-safe
@nclopezo Hi David, could you check if the jenkins comparisons with the baseline would complete. Thank you |
@slava77 |
Referring to PR cms-sw#2799, the file has been changed to make geom thread-safe.
GeometricDet::setGeographicalID is now non-const. To allow children
of a GeometricDet to have their geomgraphicalID set we added a
component(size_t) method which gives non-const access to a child.
All classes which were modifying the GeometricDet have been updated
to support the new interface.