Skip to content

Commit 427cb7e

Browse files
ValerieRamillienValerieRamillien
committed
MID:: DigitsQCCheck : Add Check for nullptr
1 parent a7315a9 commit 427cb7e

File tree

1 file changed

+74
-61
lines changed

1 file changed

+74
-61
lines changed

‎Modules/MUON/MID/src/DigitsQcCheck.cxx‎

Lines changed: 74 additions & 61 deletions
Original file line numberDiff line numberDiff line change
@@ -248,74 +248,87 @@ void DigitsQcCheck::beautify(std::shared_ptr<MonitorObject> mo, Quality checkRes
248248
if (mo->getName().find(lbHistoName) != std::string::npos) {
249249
// This matches "LocalBoardsMap*"
250250
auto histo = dynamic_cast<TH2F*>(mo->getObject());
251-
if (mo->getName() == lbHistoName) {
252-
// This is LocalBoardsMap and it was already scaled in the checker
253-
if (!checkResult.getFlags().empty()) {
254-
mHistoHelper.addLatex(histo, 0.12, 0.72, color, checkResult.getFlags().front().second.c_str());
251+
if (histo) {
252+
if (mo->getName() == lbHistoName) {
253+
// This is LocalBoardsMap and it was already scaled in the checker
254+
if (!checkResult.getFlags().empty()) {
255+
mHistoHelper.addLatex(histo, 0.12, 0.72, color, checkResult.getFlags().front().second.c_str());
256+
}
257+
mHistoHelper.addLatex(histo, 0.3, 0.32, color, fmt::format("Quality::{}", checkResult.getName()));
258+
histo->SetMaximum(zcontoursLoc4.back());
259+
histo->SetContour(zcontoursLoc4.size(), zcontoursLoc4.data());
260+
} else {
261+
mHistoHelper.normalizeHistoTokHz(histo);
262+
histo->SetMaximum(zcontoursLoc.back());
263+
histo->SetContour(zcontoursLoc.size(), zcontoursLoc.data());
255264
}
256-
mHistoHelper.addLatex(histo, 0.3, 0.32, color, fmt::format("Quality::{}", checkResult.getName()));
257-
histo->SetMaximum(zcontoursLoc4.back());
258-
histo->SetContour(zcontoursLoc4.size(), zcontoursLoc4.data());
259-
} else {
260-
mHistoHelper.normalizeHistoTokHz(histo);
261-
histo->SetMaximum(zcontoursLoc.back());
262-
histo->SetContour(zcontoursLoc.size(), zcontoursLoc.data());
265+
mHistoHelper.updateTitleWithNTF(histo);
266+
histo->SetStats(0);
263267
}
264-
mHistoHelper.updateTitleWithNTF(histo);
265-
histo->SetStats(0);
266268
} else if (mo->getName().find("BendHitsMap") != std::string::npos) { // Strips Display
267269
// This matches both [N]BendHitsMap*
268270
int maxStrip = 20; // 20kHz Max Display
269271
auto histo = dynamic_cast<TH2F*>(mo->getObject());
270-
mHistoHelper.normalizeHistoTokHz(histo);
271-
histo->SetMaximum(zcontoursStrip.back());
272-
histo->SetContour(zcontoursStrip.size(), zcontoursStrip.data());
273-
histo->SetStats(0);
274-
} else if (mo->getName() == "Hits") {
275-
auto histo = dynamic_cast<TH1F*>(mo->getObject());
276-
mHistoHelper.normalizeHistoTokHz(histo);
277-
histo->SetStats(0);
278-
}
279-
280-
else if (mo->getName() == "GBTRate") {
281-
auto histo = dynamic_cast<TH1F*>(mo->getObject());
282-
if (histo) {
283-
// if (mHistoHelper.getNTFs() > 0)
284-
mHistoHelper.normalizeHistoTokHz(histo);
285-
histo->SetMinimum(0.);
286-
TString XLabel[32] = { "5R0", "5R1", "4R0", "4R1", "1R0", "1R1", "0R0", "0R1",
287-
"2R0", "2R1", "3R0", "3R1", "7R0", "7R1", "6R0", "6R1",
288-
"5L0", "5L1", "4L0", "4L1", "1L0", "1L1", "0L0", "0L1",
289-
"2L0", "2L1", "3L0", "3L1", "7L0", "7L1", "6L0", "6L1" };
290-
for (Int_t i = 0; i < 32; ++i)
291-
histo->GetXaxis()->SetBinLabel(i + 1, XLabel[i]);
292-
histo->GetXaxis()->SetLabelSize(0.07);
293-
histo->GetXaxis()->SetLabelColor(4);
294-
}
295-
} else if (mo->getName() == "EPRate") {
296-
auto histo = dynamic_cast<TH1F*>(mo->getObject());
297272
if (histo) {
298273
mHistoHelper.normalizeHistoTokHz(histo);
299-
histo->SetMinimum(0.);
300-
histo->GetXaxis()->SetBinLabel(1, "CRU0(990)-EP0");
301-
histo->GetXaxis()->SetBinLabel(2, "CRU0(990)-EP1");
302-
histo->GetXaxis()->SetBinLabel(3, "CRU1(974)-EP0");
303-
histo->GetXaxis()->SetBinLabel(4, "CRU1(974)-EP1");
304-
histo->GetXaxis()->SetLabelSize(0.07);
305-
histo->GetXaxis()->SetLabelColor(4);
306-
}
307-
} else if (mo->getName() == "CRURate") {
308-
auto histo = dynamic_cast<TH1F*>(mo->getObject());
309-
if (histo) {
310-
mHistoHelper.normalizeHistoTokHz(histo);
311-
histo->SetMinimum(0.);
312-
histo->GetXaxis()->SetBinLabel(1, "CRU0 (990)");
313-
histo->GetXaxis()->SetBinLabel(2, "CRU1 (974)");
314-
histo->GetXaxis()->SetLabelSize(0.07);
315-
histo->GetXaxis()->SetLabelColor(4);
274+
histo->SetMaximum(zcontoursStrip.back());
275+
histo->SetContour(zcontoursStrip.size(), zcontoursStrip.data());
276+
histo->SetStats(0);
277+
{
278+
}
279+
else if (mo->getName() == "Hits")
280+
{
281+
auto histo = dynamic_cast<TH1F*>(mo->getObject());
282+
if (histo) {
283+
mHistoHelper.normalizeHistoTokHz(histo);
284+
histo->SetStats(0);
285+
}
286+
}
287+
288+
else if (mo->getName() == "GBTRate")
289+
{
290+
auto histo = dynamic_cast<TH1F*>(mo->getObject());
291+
if (histo) {
292+
// if (mHistoHelper.getNTFs() > 0)
293+
mHistoHelper.normalizeHistoTokHz(histo);
294+
histo->SetMinimum(0.);
295+
TString XLabel[32] = { "5R0", "5R1", "4R0", "4R1", "1R0", "1R1", "0R0", "0R1",
296+
"2R0", "2R1", "3R0", "3R1", "7R0", "7R1", "6R0", "6R1",
297+
"5L0", "5L1", "4L0", "4L1", "1L0", "1L1", "0L0", "0L1",
298+
"2L0", "2L1", "3L0", "3L1", "7L0", "7L1", "6L0", "6L1" };
299+
for (Int_t i = 0; i < 32; ++i)
300+
histo->GetXaxis()->SetBinLabel(i + 1, XLabel[i]);
301+
histo->GetXaxis()->SetLabelSize(0.07);
302+
histo->GetXaxis()->SetLabelColor(4);
303+
}
304+
}
305+
else if (mo->getName() == "EPRate")
306+
{
307+
auto histo = dynamic_cast<TH1F*>(mo->getObject());
308+
if (histo) {
309+
mHistoHelper.normalizeHistoTokHz(histo);
310+
histo->SetMinimum(0.);
311+
histo->GetXaxis()->SetBinLabel(1, "CRU0(990)-EP0");
312+
histo->GetXaxis()->SetBinLabel(2, "CRU0(990)-EP1");
313+
histo->GetXaxis()->SetBinLabel(3, "CRU1(974)-EP0");
314+
histo->GetXaxis()->SetBinLabel(4, "CRU1(974)-EP1");
315+
histo->GetXaxis()->SetLabelSize(0.07);
316+
histo->GetXaxis()->SetLabelColor(4);
317+
}
318+
}
319+
else if (mo->getName() == "CRURate")
320+
{
321+
auto histo = dynamic_cast<TH1F*>(mo->getObject());
322+
if (histo) {
323+
mHistoHelper.normalizeHistoTokHz(histo);
324+
histo->SetMinimum(0.);
325+
histo->GetXaxis()->SetBinLabel(1, "CRU0 (990)");
326+
histo->GetXaxis()->SetBinLabel(2, "CRU1 (974)");
327+
histo->GetXaxis()->SetLabelSize(0.07);
328+
histo->GetXaxis()->SetLabelColor(4);
329+
}
330+
}
316331
}
332+
mHistoHelper.updateTitle(dynamic_cast<TH1*>(mo->getObject()), mHistoHelper.getCurrentTime());
317333
}
318-
}
319-
mHistoHelper.updateTitle(dynamic_cast<TH1*>(mo->getObject()), mHistoHelper.getCurrentTime());
320-
}
321-
} // namespace o2::quality_control_modules::mid
334+
} // namespace o2::quality_control_modules::mid

0 commit comments

Comments
 (0)