Skip to content

Commit 35ad896

Browse files
author
Sven Dildick
committed
Fix unit test
1 parent 92eb7e3 commit 35ad896

File tree

2 files changed

+22
-19
lines changed

2 files changed

+22
-19
lines changed

DataFormats/CSCDigi/interface/CSCCorrelatedLCTDigi.h

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,18 @@ class CSCCorrelatedLCTDigi {
2222
// for data vs emulator studies
2323
enum LCTBXMask { kBXDataMask = 0x1 };
2424

25+
/// SIMULATION ONLY ////
26+
enum Type {
27+
CLCTALCT, // CLCT-centric
28+
ALCTCLCT, // ALCT-centric
29+
ALCTCLCTGEM, // ALCT-CLCT-1 GEM pad
30+
ALCTCLCT2GEM, // ALCT-CLCT-2 GEM pads in coincidence
31+
ALCT2GEM, // ALCT-2 GEM pads in coincidence
32+
CLCT2GEM, // CLCT-2 GEM pads in coincidence
33+
CLCTONLY, // Missing ALCT
34+
ALCTONLY // Missing CLCT
35+
};
36+
2537
/// Constructors
2638
CSCCorrelatedLCTDigi(const uint16_t trknmb,
2739
const uint16_t valid,
@@ -39,7 +51,8 @@ class CSCCorrelatedLCTDigi {
3951
const bool run3_quart_strip_bit = false,
4052
const bool run3_eighth_strip_bit = false,
4153
const uint16_t run3_pattern = 0,
42-
const uint16_t run3_slope = 0);
54+
const uint16_t run3_slope = 0,
55+
const int type = ALCTCLCT);
4356

4457
/// default (calls clear())
4558
CSCCorrelatedLCTDigi();
@@ -195,18 +208,6 @@ class CSCCorrelatedLCTDigi {
195208

196209
void setRun3(const bool isRun3);
197210

198-
/// SIMULATION ONLY ////
199-
enum Type {
200-
CLCTALCT, // CLCT-centric
201-
ALCTCLCT, // ALCT-centric
202-
ALCTCLCTGEM, // ALCT-CLCT-1 GEM pad
203-
ALCTCLCT2GEM, // ALCT-CLCT-2 GEM pads in coincidence
204-
ALCT2GEM, // ALCT-2 GEM pads in coincidence
205-
CLCT2GEM, // CLCT-2 GEM pads in coincidence
206-
CLCTONLY, // Missing ALCT
207-
ALCTONLY // Missing CLCT
208-
};
209-
210211
int getType() const { return type_; }
211212

212213
void setType(int type) { type_ = type; }

DataFormats/CSCDigi/src/CSCCorrelatedLCTDigi.cc

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ CSCCorrelatedLCTDigi::CSCCorrelatedLCTDigi(const uint16_t itrknmb,
2727
const bool run3_quart_strip_bit,
2828
const bool run3_eighth_strip_bit,
2929
const uint16_t run3_pattern,
30-
const uint16_t run3_slope)
30+
const uint16_t run3_slope,
31+
const int type)
3132
: trknmb(itrknmb),
3233
valid(ivalid),
3334
quality(iquality),
@@ -40,10 +41,12 @@ CSCCorrelatedLCTDigi::CSCCorrelatedLCTDigi(const uint16_t itrknmb,
4041
bx0(ibx0),
4142
syncErr(isyncErr),
4243
cscID(icscID),
44+
hmt(0),
4345
run3_quart_strip_bit_(run3_quart_strip_bit),
4446
run3_eighth_strip_bit_(run3_eighth_strip_bit),
4547
run3_pattern_(run3_pattern),
4648
run3_slope_(run3_slope),
49+
type_(type),
4750
version_(version) {}
4851

4952
/// Default
@@ -73,7 +76,7 @@ void CSCCorrelatedLCTDigi::clear() {
7376
run3_pattern_ = 0;
7477
run3_slope_ = 0;
7578
// clear the components
76-
type_ = 0;
79+
type_ = 1;
7780
alct_.clear();
7881
clct_.clear();
7982
gem1_ = GEMPadDigi();
@@ -133,7 +136,7 @@ void CSCCorrelatedLCTDigi::setRun3(const bool isRun3) { version_ = isRun3 ? Vers
133136
bool CSCCorrelatedLCTDigi::operator==(const CSCCorrelatedLCTDigi& rhs) const {
134137
return ((trknmb == rhs.trknmb) && (quality == rhs.quality) && (keywire == rhs.keywire) && (strip == rhs.strip) &&
135138
(pattern == rhs.pattern) && (bend == rhs.bend) && (bx == rhs.bx) && (valid == rhs.valid) &&
136-
(mpclink == rhs.mpclink) && (hmt == rhs.hmt));
139+
(mpclink == rhs.mpclink));
137140
}
138141

139142
/// Debug
@@ -143,8 +146,7 @@ void CSCCorrelatedLCTDigi::print() const {
143146
<< " Quality = " << getQuality() << " Key Wire = " << getKeyWG()
144147
<< " Strip = " << getStrip() << " Pattern = " << getPattern()
145148
<< " Bend = " << ((getBend() == 0) ? 'L' : 'R') << " BX = " << getBX()
146-
<< " MPC Link = " << getMPCLink() << " Type (SIM) = " << getType()
147-
<< " HMT Bit = " << getHMT();
149+
<< " MPC Link = " << getMPCLink() << " Type (SIM) = " << getType();
148150
} else {
149151
edm::LogVerbatim("CSCDigi") << "Not a valid correlated LCT.";
150152
}
@@ -153,7 +155,7 @@ void CSCCorrelatedLCTDigi::print() const {
153155
std::ostream& operator<<(std::ostream& o, const CSCCorrelatedLCTDigi& digi) {
154156
return o << "CSC LCT #" << digi.getTrknmb() << ": Valid = " << digi.isValid() << " Quality = " << digi.getQuality()
155157
<< " MPC Link = " << digi.getMPCLink() << " cscID = " << digi.getCSCID()
156-
<< " syncErr = " << digi.getSyncErr() << " Type (SIM) = " << digi.getType() << " HMT Bit = " << digi.getHMT()
158+
<< " syncErr = " << digi.getSyncErr() << " Type (SIM) = " << digi.getType()
157159
<< "\n"
158160
<< " cathode info: Strip = " << digi.getStrip() << " Pattern = " << digi.getPattern()
159161
<< " Bend = " << ((digi.getBend() == 0) ? 'L' : 'R') << "\n"

0 commit comments

Comments
 (0)