Update KVertex

The fake member is not needed as it can be calculated form other members.
The valid member is added.

Author: JoramBerger

DataFormats/interface/KBasic.h

@@ -41,12 +41,14 @@ typedef std::vector<KBeamSpot> KBeamSpots;
 struct KVertex
 {
 	RMPoint position;
-	bool fake;
-
+	bool valid;
 	unsigned int nTracks;
-	float chi2, nDOF;
+	float chi2;
+	float nDOF;
+
+	ROOT::Math::SMatrix<double, 3, 3, ROOT::Math::MatRepSym<double, 3> > covariance;  //< covariance matrix
 
-	ROOT::Math::SMatrix<double, 3, 3, ROOT::Math::MatRepSym<double, 3> > covariance;
+	inline bool fake() const { return (chi2 == 0 && nDOF == 0 && nTracks == 0); };
 };
 typedef std::vector<KVertex> KVertices;
 

DataFormats/test/KDebug.cpp

@@ -91,7 +91,7 @@ std::ostream &operator<<(std::ostream &os, const KGenTau &tau)
 std::ostream &operator<<(std::ostream &os, const KVertex &vertex)
 {
 	return os << vertex.position
-		<< " #Tracks=" << vertex.nTracks << " fake=" << vertex.fake
+		<< " #Tracks=" << vertex.nTracks << " valid=" << vertex.valid
 		<< " chi2=" << vertex.chi2 << " DOF=" << vertex.nDOF;
 }
 

Producers/interface/KVertexProducer.h

@@ -22,7 +22,7 @@ class KVertexProducer : public KBaseMultiVectorProducer<edm::View<reco::Vertex>,
 	static void fillVertex(const SingleInputType &in, SingleOutputType &out)
 	{
 		out.position = in.position();
-		out.fake = in.isFake();
+		out.valid = in.isValid();
 		out.nTracks = in.tracksSize();
 
 		out.chi2 = in.chi2();