Tracks and TrackStates should not use Tracking coordinates
The serialized objects should not use the tracking coordinate system which is specific only to Tracking and ACTS. In order to make the objects more usable by others in LDMX, the data should be stored in detector coordinates (like everything else) and the conversion to/from tracking coordinates should stay within the Tracking module.
Is this not the case now?
Nope! I was looking at some recoil tracks of beam electrons and observed that (on average) momentum = (8, 0, 0) from this I interpret that we are still in tracking coordinates and using GeV as the energy unit.
Shoot...I'll fix this.