GRIFFINCollaboration / detectorSimulations

GEANT4 simulation code for the GRIFFIN array and it's suite of ancillary detection systems.
7 stars 13 forks source link

bug fixes #76

Closed ghost closed 10 years ago

ghost commented 10 years ago

DetectorConstruction: The world volume was originally made with air, this caused very few full-energy electron detections. TechVacuum itself is too high pressure and will be changed in the suppressed files after investigation (it is currently unused elsewhere in the program)

EventAction: SortEvent() brought outside the step loop, so that energy per step deposit can be summed accordingly.

PrimGen: There was no direction involved in the energyrange function that I initially put into the simulation.

evan012345 commented 10 years ago

@evitts There is a messenger command in DetectorMessenger.cc to change the world material to anything you want during runtime, eg. /DetSys/world/material G4_Water. I would suggest leaving the default as air as this is usually what we will want when testing the GRIFFIN detectors.

ghost commented 10 years ago

Ah, thank you. I've changed it back.
I'll have a look through the messenger files today and write a glossary of available commands.

evan012345 commented 10 years ago

@evitts Sounds like a great idea! Also add whatever messenger commands you need; they keep the code from being too hard-coded.

bkatiemills commented 10 years ago

Yes. I approve of all of this - a glossary of commands would be just about the best thing possible right now. @evan012345, thanks for being so on top of this!