Skip to content
Snippets Groups Projects

Make add virtual and add to common base so that both GeoPhysVol and GeoFullPhysVol can be used as envelopes

Merged Nicholas Styles requested to merge nstyles/GeoModel:master-GeoVPhysVolUpdate into master
All threads resolved!

Follow-up from !141 (closed)

When investigating the possibility to use the existing GeoModelXml "Envelope" mechanism to insert new service material inside an existing volume (in this case to position ITk/HGTD service material in the Calorimeter crack region) it was noticed that the mechanism only works for GeoPhysVols while the specific location where new items would ideally be inserted is a GeoFullPhysVol. This MR adds the capability to also use a GeoFullPhysVol as an Envelope, through making the add(GeoGraphNode* graphNode) method a virtual method of the common base class used by both GeoPhysVol and GeoFullPhysVol, i.e. GeoVPhysVol.

FYI @boudreau @mbandier @dellacqu

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
Please register or sign in to reply
Loading