Skip to content

Commit 972ba34

Browse files
awegrzynBarthelemy
authored andcommitted
monitoring lowercase namespace (#8)
1 parent 40ac89f commit 972ba34

File tree

5 files changed

+24
-25
lines changed

5 files changed

+24
-25
lines changed

Framework/include/QualityControl/Checker.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ class Checker : public FairMQDevice
101101
std::map<std::string,TClass*> mClassesLoaded;
102102

103103
// monitoring
104-
std::shared_ptr<AliceO2::Monitoring::Collector> mCollector;
104+
std::shared_ptr<o2::monitoring::Monitoring> mCollector;
105105
std::chrono::system_clock::time_point startFirstObject;
106106
std::chrono::system_clock::time_point endLastObject;
107107
int mTotalNumberHistosReceived = 0;

Framework/include/QualityControl/TaskDevice.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ class TaskDevice : public FairMQDevice
6262
std::string mTaskName;
6363
TaskConfig mTaskConfig;
6464
std::unique_ptr<AliceO2::Configuration::ConfigurationInterface> mConfigFile;
65-
std::unique_ptr<AliceO2::Monitoring::Collector> mCollector;
65+
std::unique_ptr<o2::monitoring::Monitoring> mCollector;
6666
std::unique_ptr<AliceO2::DataSampling::SamplerInterface> mSampler;
6767
o2::quality_control::core::TaskInterface *mTask;
6868
std::shared_ptr<ObjectsManager> mObjectsManager;

Framework/src/Checker.cxx

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ using namespace std;
3737
using namespace AliceO2::InfoLogger;
3838
using namespace std::chrono;
3939
using namespace AliceO2::Configuration;
40-
using namespace AliceO2::Monitoring;
40+
using namespace o2::monitoring;
4141

4242
namespace o2 {
4343
namespace quality_control {
@@ -68,7 +68,6 @@ Checker::Checker(std::string checkerName, std::string configurationSource)
6868
// monitoring
6969
try {
7070
mCollector = MonitoringFactory::Get("infologger://");
71-
mCollector->addDerivedMetric("objects", AliceO2::Monitoring::DerivedMetricMode::RATE);
7271
} catch (...) {
7372
string diagnostic = boost::current_exception_diagnostic_information();
7473
std::cerr << "Unexpected exception, diagnostic information follows:\n" << diagnostic << endl;
@@ -132,10 +131,10 @@ Checker::~Checker()
132131

133132
// Monitoring
134133
std::chrono::duration<double> diff = endLastObject - startFirstObject;
135-
mCollector->send(diff.count(), "QC_checker_Time_between_first_and_last_objects_received");
136-
mCollector->send(mTotalNumberHistosReceived, "QC_checker_Total_number_histos_treated");
134+
mCollector->send({diff.count(), "QC_checker_Time_between_first_and_last_objects_received"});
135+
mCollector->send({mTotalNumberHistosReceived, "QC_checker_Total_number_histos_treated"});
137136
double rate = mTotalNumberHistosReceived / diff.count();
138-
mCollector->send(rate, "QC_checker_Rate_objects_treated_per_second_whole_run");
137+
mCollector->send({rate, "QC_checker_Rate_objects_treated_per_second_whole_run"});
139138
}
140139

141140
int size_t2int(size_t val)
@@ -173,7 +172,7 @@ bool Checker::HandleData(FairMQMessagePtr &msg, int index)
173172
// if 10 seconds elapsed publish stats
174173
if (timer.isTimeout()) {
175174
timer.reset(1000000); // 10 s.
176-
mCollector->send(mTotalNumberHistosReceived, "objects");
175+
mCollector->send({mTotalNumberHistosReceived, "objects"}, o2::monitoring::DerivedMetricMode::RATE);
177176
}
178177

179178
return true; // keep running

Framework/src/TaskDevice.cxx

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ namespace bpo = boost::program_options;
1919
using namespace std;
2020
using namespace std::chrono;
2121
using namespace AliceO2::Configuration;
22-
using namespace AliceO2::Monitoring;
22+
using namespace o2::monitoring;
2323

2424
/*
2525
* Runner
@@ -129,7 +129,7 @@ void TaskDevice::Run()
129129
double current = timer.getTime();
130130
int objectsPublished = (mTotalNumberObjectsPublished-lastNumberObjects);
131131
lastNumberObjects = mTotalNumberObjectsPublished;
132-
mCollector->send(objectsPublished/current, "QC_task_Rate_objects_published_per_10_seconds");
132+
mCollector->send({objectsPublished/current, "QC_task_Rate_objects_published_per_10_seconds"});
133133
timer.increment();
134134
}
135135
}
@@ -163,23 +163,23 @@ void TaskDevice::monitorCycle()
163163
double durationCycle = timer.getTime();
164164
timer.reset();
165165
double durationPublication = timer.getTime();
166-
mCollector->send(numberBlocks, "QC_task_Numberofblocks_in_cycle");
167-
mCollector->send(durationCycle, "QC_task_Module_cycle_duration");
168-
mCollector->send(durationPublication, "QC_task_Publication_duration");
169-
mCollector->send((int) numberObjectsPublished,
170-
"QC_task_Number_objects_published_in_cycle"); // cast due to Monitoring accepting only int
166+
mCollector->send({numberBlocks, "QC_task_Numberofblocks_in_cycle"});
167+
mCollector->send({durationCycle, "QC_task_Module_cycle_duration"});
168+
mCollector->send({durationPublication, "QC_task_Publication_duration"});
169+
mCollector->send({(int) numberObjectsPublished,
170+
"QC_task_Number_objects_published_in_cycle"}); // cast due to Monitoring accepting only int
171171
double rate = numberObjectsPublished / (durationCycle + durationPublication);
172-
mCollector->send(rate, "QC_task_Rate_objects_published_per_second");
172+
mCollector->send({rate, "QC_task_Rate_objects_published_per_second"});
173173
mTotalNumberObjectsPublished += numberObjectsPublished;
174174
//std::vector<std::string> pidStatus = mMonitor->getPIDStatus(::getpid());
175175
//pcpus(std::stod(pidStatus[3]));
176176
//pmems(std::stod(pidStatus[4]));
177177
double whole_run_rate = mTotalNumberObjectsPublished / timerTotalDurationActivity.getTime();
178-
mCollector->send(mTotalNumberObjectsPublished, "QC_task_Total_objects_published_whole_run");
179-
mCollector->send(timerTotalDurationActivity.getTime(), "QC_task_Total_duration_activity_whole_run");
180-
mCollector->send(whole_run_rate, "QC_task_Rate_objects_published_per_second_whole_run");
181-
// mCollector->send(std::stod(pidStatus[3]), "QC_task_Mean_pcpu_whole_run");
182-
mCollector->send(ba::mean(pmems), "QC_task_Mean_pmem_whole_run");
178+
mCollector->send({mTotalNumberObjectsPublished, "QC_task_Total_objects_published_whole_run"});
179+
mCollector->send({timerTotalDurationActivity.getTime(), "QC_task_Total_duration_activity_whole_run"});
180+
mCollector->send({whole_run_rate, "QC_task_Rate_objects_published_per_second_whole_run"});
181+
// mCollector->send({std::stod(pidStatus[3]), "QC_task_Mean_pcpu_whole_run"});
182+
mCollector->send({ba::mean(pmems), "QC_task_Mean_pmem_whole_run"});
183183
}
184184

185185
unsigned long TaskDevice::publish()
@@ -228,9 +228,9 @@ void TaskDevice::endOfActivity()
228228
mTask->endOfActivity(activity);
229229

230230
double rate = mTotalNumberObjectsPublished / timerTotalDurationActivity.getTime();
231-
mCollector->send(rate, "QC_task_Rate_objects_published_per_second_whole_run");
232-
mCollector->send(ba::mean(pcpus), "QC_task_Mean_pcpu_whole_run");
233-
mCollector->send(ba::mean(pmems), "QC_task_Mean_pmem_whole_run");
231+
mCollector->send({rate, "QC_task_Rate_objects_published_per_second_whole_run"});
232+
mCollector->send({ba::mean(pcpus), "QC_task_Mean_pcpu_whole_run"});
233+
mCollector->send({ba::mean(pmems), "QC_task_Mean_pmem_whole_run"});
234234
}
235235

236236
void TaskDevice::sendToInformationService(string objectsListString)

cmake/FindMonitoring.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
include(FindPackageHandleStandardArgs)
2121

2222
# find includes
23-
find_path(MONITORING_INCLUDE_DIR Collector.h
23+
find_path(MONITORING_INCLUDE_DIR Monitoring.h
2424
HINTS ${Monitoring_ROOT}/include ENV LD_LIBRARY_PATH PATH_SUFFIXES "../include/Monitoring" "../../include/Monitoring" )
2525

2626
# Remove the final "Monitoring"

0 commit comments

Comments
 (0)