USER: Fixed particle ID issues in TA2GoAT. Calling E from the particle class is the total E (including mass), when particle ID cuts have been turned on, this will cause reconstruction errors. particle is copied over, mass is set to zero (to fix 4 vectors) and T is called instead of E. This lead to the discovery of another issue: TAPS will call any charged particle a proton if it does not have a dE_E cut to check with. We have a Rootino code for a reason! So I've changed it to call a charged particle a Rootino in the case that no dE_E cut is supplied. Also some issues fixed inside of TA2AccessSQL which calls the fNaI without any checks of whether the NaI are active.
USER: Beginning to make the switch to E, theta, phi to deal with the following issue - when Px, Py, Pz, E are output from AcquRoot (which is before Particle ID has been done) the Px,Py,Pz, E has a zero mass assumed but this isn't correct! Once particle ID has been done these values can be stored (ie. from GoAT) but before this, only kinetic energy, theta and phi should be output as they will not change. In a future commit, I will change E to T for improved clarity on this issue. Resulting spectra from this step look reasonable
USER: Switch to a raw tree file based on E-theta-phi (instead of E-Px-Py-Pz) is now complete. This move was required as the momentum components will only be accurate post particle identification.
USER: Small changes to config files, pushing to remote for sharing purposes of recent (local only) commits