-
Notifications
You must be signed in to change notification settings - Fork 35
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
Ensure CMS works for Run3 and HL-LHC, through GMDL and CMSSW #822
Comments
With manually adding HL-LHC sensitive detectors:
|
The log file with all calorimeter SDs added is test_find_volumes_many.log. |
JSON diagnostic output file with a calorimeter-sd-only run: |
OK, even though the "find volume" is mapping e.g. Could something be changing the names between the time that we call Even weirder is that these names seem to disagree only for the |
List of all CMS CaloSD volumes from LH-LHC: cms-hllhc-CaloSD-volumes.txt |
@whokion I've added an updated list of the reachable volumes with attached sensitive detectors To recap: our best guess is that CMSSW is attaching SDs to volumes that are not part of the world volume hierarchy: this only seems to be happening to the |
Back trace for the segmentation fault with all CMS HLHLC tracker sensitive volumes:
|
The clean list cleared the exception, but hits another at HGCalSD due to missing information:
|
Here's the list of G4VSD daughter classes in CMS that I can find, with the ones we don't need commented out:
And as far as I can tell here are the properties, but I haven't done anything to exclude the debug statements other than filter out code with
Extracting uses$ grep -Hnr '\(StepPoint\|Track|Step\)()->Get' Calo Tracker | sort # Vim: g/<Get([^;, ]+).* # \1 \2->Get\3 All usesCalo/plugins/CaloSteppingAction.cc:335: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume() Calo/plugins/CaloSteppingAction.cc:346: PreStepPoint->GetTouchable() Calo/plugins/CaloSteppingAction.cc:347: PreStepPoint->GetPosition() Calo/plugins/CaloSteppingAction.cc:356: PreStepPoint->GetCharge() Calo/plugins/CaloSteppingAction.cc:357: PreStepPoint->GetMaterial()->GetDensity() Calo/plugins/CaloSteppingAction.cc:378: PreStepPoint->GetMaterial()->GetRadlen() Calo/plugins/CaloSteppingAction.cc:383: PreStepPoint->GetCharge() Calo/plugins/CaloSteppingAction.cc:384: PreStepPoint->GetMaterial()->GetDensity() Calo/plugins/CaloSteppingAction.cc:395: PreStepPoint->GetTouchable() Calo/plugins/CaloSteppingAction.cc:396: Track->GetGlobalTime() Calo/plugins/CaloSteppingAction.cc:397: Track->GetTrackID() Calo/plugins/CaloSteppingAction.cc:398: Track->GetDefinition()->GetPDGEncoding() Calo/plugins/CaloSteppingAction.cc:400: PreStepPoint->GetPosition().x() Calo/plugins/CaloSteppingAction.cc:401: PreStepPoint->GetPosition().y() Calo/plugins/CaloSteppingAction.cc:402: PreStepPoint->GetPosition().z() Calo/plugins/CaloSteppingAction.cc:411: Track->GetNextVolume() Calo/plugins/CaloSteppingAction.cc:413: PreStepPoint->GetKineticEnergy() Calo/plugins/CaloSteppingAction.cc:415: PostStepPoint->GetGlobalTime() Calo/plugins/CaloSteppingAction.cc:435: Track->GetPosition() Calo/plugins/CaloSteppingAction.cc:437: Track->GetMomentum() Calo/plugins/CaloSteppingAction.cc:441: PreStepPoint->GetPhysicalVolume() Calo/plugins/HcalTestAnalysis.cc:313: PreStepPoint->GetPhysicalVolume() Calo/plugins/HcalTestAnalysis.cc:351: Track->GetDefinition()->GetParticleName() Calo/plugins/HcalTestAnalysis.cc:353: PreStepPoint->GetPosition().x() Calo/plugins/HcalTestAnalysis.cc:354: PreStepPoint->GetPosition().y() Calo/plugins/HcalTestAnalysis.cc:355: PreStepPoint->GetPosition().z()) Calo/src/CaloSD.cc:172: PreStepPoint->GetTouchable())) Calo/src/CaloSD.cc:183: PreStepPoint->GetPhysicalVolume() Calo/src/CaloSD.cc:269: Track->GetPrimaryTrack() Calo/src/CaloSD.cc:436: PreStepPoint->GetPosition() Calo/src/CaloSD.cc:678: PreStepPoint->GetCharge() Calo/src/CaloSD.cc:682: PreStepPoint->GetMaterial()->GetDensity() Calo/src/CaloTrkProcessing.cc:185: PreStepPoint->GetTouchable() Calo/src/CaloTrkProcessing.cc:186: PostStepPoint->GetTouchable() Calo/src/ECalSD.cc:457: PreStepPoint->GetTouchable() Calo/src/HCalSD.cc:337: PreStepPoint->GetTouchable()->GetReplicaNumber(0)) Calo/src/HCalSD.cc:350: PreStepPoint->GetPosition() Calo/src/HCalSD.cc:374: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume()->GetName() Calo/src/HCalSD.cc:382: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume()->GetName() Calo/src/HCalSD.cc:401: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume() Calo/src/HCalSD.cc:453: PreStepPoint->GetTouchable() Calo/src/HCalSD.cc:476: PreStepPoint->GetPosition() Calo/src/HCalSD.cc:515: PreStepPoint->GetMaterial() Calo/src/HCalSD.cc:634: PreStepPoint->GetTouchable() Calo/src/HCalSD.cc:868: PostStepPoint->GetGlobalTime()) Calo/src/HCalSD.cc:927: PostStepPoint->GetGlobalTime()) Calo/src/HCalSD.cc:999: PreStepPoint->GetTouchable() Calo/src/HFCherenkov.cc:65: Track->GetWeight() Calo/src/HFCherenkov.cc:77: PreStepPoint->GetPosition()) Calo/src/HFCherenkov.cc:79: PostStepPoint->GetPosition()) Calo/src/HFGflash.cc:155: PreStepPoint->GetCharge() Calo/src/HFNoseSD.cc:103: Track->GetWeight() Calo/src/HFNoseSD.cc:139: PreStepPoint->GetMaterial() Calo/src/HFNoseSD.cc:150: PreStepPoint->GetMaterial()->GetRadlen() Calo/src/HFNoseSD.cc:84: PreStepPoint->GetPosition().perp() Calo/src/HFNoseSD.cc:85: PreStepPoint->GetPosition().z()) Calo/src/HFNoseSD.cc:87: Track->GetDefinition()->GetPDGEncoding() Calo/src/HFNoseSD.cc:88: Track->GetDefinition()->GetParticleName() Calo/src/HFNoseSD.cc:89: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume() Calo/src/HFShower.cc:111: PostStepPoint->GetGlobalTime()) Calo/src/HFShower.cc:180: Track->GetDefinition()->GetPDGCharge() Calo/src/HFShower.cc:220: PostStepPoint->GetGlobalTime()) Calo/src/HFShower.cc:269: Track->GetDefinition()->GetPDGCharge() Calo/src/HFShower.cc:329: PostStepPoint->GetGlobalTime()) Calo/src/HFShower.cc:51: Track->GetDefinition()->GetPDGCharge() Calo/src/HFShowerParam.cc:302: PostStepPoint->GetGlobalTime()) Calo/src/HGCSD.cc:102: Track->GetDefinition()->GetPDGEncoding() Calo/src/HGCSD.cc:103: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume() Calo/src/HGCSD.cc:116: Track->GetWeight() Calo/src/HGCSD.cc:122: PreStepPoint->GetTouchable() Calo/src/HGCSD.cc:123: Track->GetKineticEnergy() Calo/src/HGCSD.cc:124: Track->GetDefinition()->GetPDGEncoding() Calo/src/HGCSD.cc:125: Track->GetMomentumDirection().theta()) Calo/src/HGCSD.cc:170: PreStepPoint->GetMaterial() Calo/src/HGCSD.cc:183: PreStepPoint->GetMaterial()->GetRadlen() Calo/src/HGCSD.cc:98: PreStepPoint->GetPosition().perp() Calo/src/HGCSD.cc:99: PreStepPoint->GetPosition().z()) Calo/src/HGCScintSD.cc:119: PreStepPoint->GetPosition().perp() Calo/src/HGCScintSD.cc:120: PreStepPoint->GetPosition().z()) Calo/src/HGCScintSD.cc:122: Track->GetDefinition()->GetPDGEncoding() Calo/src/HGCScintSD.cc:123: Track->GetDefinition()->GetParticleName() Calo/src/HGCScintSD.cc:124: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume() Calo/src/HGCScintSD.cc:138: Track->GetWeight() Calo/src/HGCScintSD.cc:185: PreStepPoint->GetMaterial() Calo/src/HGCScintSD.cc:196: PreStepPoint->GetMaterial()->GetRadlen() Calo/src/HGCalSD.cc:101: PreStepPoint->GetPosition().perp() Calo/src/HGCalSD.cc:102: PreStepPoint->GetPosition().z()) Calo/src/HGCalSD.cc:104: Track->GetDefinition()->GetPDGEncoding() Calo/src/HGCalSD.cc:105: Track->GetDefinition()->GetParticleName() Calo/src/HGCalSD.cc:106: PreStepPoint->GetPhysicalVolume()->GetLogicalVolume() Calo/src/HGCalSD.cc:120: Track->GetWeight() Calo/src/HGCalSD.cc:162: PreStepPoint->GetMaterial() Calo/src/HGCalSD.cc:173: PreStepPoint->GetMaterial()->GetRadlen() Tracker/src/TkAccumulatingSensitiveDetector.cc:123: PreStepPoint->GetTouchable()) Tracker/src/TkAccumulatingSensitiveDetector.cc:281: PostStepPoint->GetPosition()) Tracker/src/TkAccumulatingSensitiveDetector.cc:305: PostStepPoint->GetPosition()) Notes:
|
@whokion confirms that we can run CMSSW:
Yay! Now we just have to get it to run fast... |
CMS Run 3 geometry
20 volumes listed below are failing
G4UTrap::CheckPlanarity()
(both Side face ~-X and Side face ~+X and ) with cms-run3.gdml. For the moment, @whokion by-passes this exception with downgrading the severity to JustWarning in Geant4:CMS HL-LHC
ECalSD::getBaseNumber
fixed bysd.locate_touchable
andsd.pre.position
SensitiveDetector::cmsTrackInformation(const G4Track* aTrack)
: CMSSW has a NewTrackAction which calls something which calls callsG4Track::SetUserInformation
; the NewTrackAction hooks into a daughter ofG4UserStackingAction
The text was updated successfully, but these errors were encountered: