alcap-org / g4sim

Simulation toolkit based on Geant4 and ROOT
http://wuchen1106.github.io/g4sim/
2 stars 2 forks source link

Anisotropic bug fix + other updates #58

Closed jrquirk closed 5 years ago

jrquirk commented 5 years ago

Pay special attention to PrimaryGeneratorAction.cc. I added a some stuff but mainly the SetUniformDirection and SetRandomDirection should be checked.

AndrewEdmonds11 commented 5 years ago

Looks good to me! I would still opt to remove SetUniformDirection since you can always use SetRandomDirection with the uRand option and appropriate theta and phi spreads to do the same thing.

jrquirk commented 5 years ago

I think you're right. Two options that do the same thing is confusing.