Draft: Resolve "VectorXYZT::mag() may be confusing since not four-vector magnitude"

Closes #19

assigned to @wdconinc

added 1 commit

• 2235002d - VectorXYZT::mass2

Compare with previous version

added 1 commit

• 268fc174 - VectorXYZ::mag2(), VectorPolar::mag2(), VectorXYZT(VectorXYZ,double), VectorXYZT::mag2()

Compare with previous version

mentioned in issue #18

From a user perspective, this XYZT object is confusing. I was going to use it to encode the track parameterizations in my private data model, but then realized indeed that it specifically implies a Minkowski metric, and opted to stay away from it, resorting to a pair of separate {XYZ} & {T} fields. In my view this VectorXYZ should better be renamed into LorentzVector, for clarity.

I agree. There are some improvements to be made to some of the naming.

I think this should be a fairly painless migration, as we're not really using the 4-vectors right now.

added 1 commit

• 0ba3071c - VectorXYZT and LorentzVectorXYZT are different types

Compare with previous version

closed