Skip to content

Commit ea2a224

Browse files
pillotalcaliva
authored andcommitted
[MCH] use DsIndex type for DS indices (#12883)
1 parent 39014ed commit ea2a224

File tree

11 files changed

+25
-21
lines changed

11 files changed

+25
-21
lines changed

Detectors/MUON/MCH/GlobalMapping/include/MCHGlobalMapping/Mapper.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
namespace o2::mch::dcs
2323
{
2424
/** get the list of dual sampa indices corresponding to a given DCS Alias */
25-
std::set<int> aliasToDsIndices(std::string_view alias);
25+
std::set<DsIndex> aliasToDsIndices(std::string_view alias);
2626

2727
/** get indices of all dual sampas of a set of cathodes (={deId,plane}).
2828
* returned set might be empty if input Cathodes are not valid ones. */

Detectors/MUON/MCH/GlobalMapping/src/HV.cxx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,6 @@
1414
#include "MCHConditions/Chamber.h"
1515
#include "MCHConditions/DetectionElement.h"
1616
#include "MCHConditions/Number.h"
17-
#include "MCHGlobalMapping/DsIndex.h"
1817
#include "MCHMappingInterface/Segmentation.h"
1918
#include "MCHRawElecMap/DsDetId.h"
2019
#include "MCHRawElecMap/Mapper.h"
@@ -37,13 +36,13 @@ double findXmean(const o2::mch::mapping::Segmentation& seg, int dsId)
3736
return xmean / n;
3837
}
3938

40-
std::set<int> hvAliasToDsIndices(std::string_view alias)
39+
std::set<DsIndex> hvAliasToDsIndices(std::string_view alias)
4140
{
4241
const auto chamber = aliasToChamber(alias);
4342
bool slat = isSlat(chamber);
4443
int deId = aliasToDetElemId(alias).value();
45-
std::set<int> indices;
46-
for (auto dsIndex = 0; dsIndex < o2::mch::NumberOfDualSampas; ++dsIndex) {
44+
std::set<DsIndex> indices;
45+
for (DsIndex dsIndex = 0; dsIndex < o2::mch::NumberOfDualSampas; ++dsIndex) {
4746
const auto& dd = getDsDetId(dsIndex);
4847
if (dd.deId() != deId) {
4948
continue;

Detectors/MUON/MCH/GlobalMapping/src/HV.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,11 @@
1515
#include <set>
1616
#include <string>
1717

18+
#include "MCHGlobalMapping/DsIndex.h"
19+
1820
namespace o2::mch::dcs
1921
{
20-
std::set<int> hvAliasToDsIndices(std::string_view alias);
22+
std::set<DsIndex> hvAliasToDsIndices(std::string_view alias);
2123
}
2224

2325
#endif

Detectors/MUON/MCH/GlobalMapping/src/LV.cxx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
#include "MCHConditions/Chamber.h"
1616
#include "MCHConditions/Plane.h"
1717
#include "MCHConstants/DetectionElements.h"
18-
#include "MCHGlobalMapping/DsIndex.h"
1918
#include "MCHRawElecMap/Mapper.h"
2019
#include "Quadrant.h"
2120
#include "Slat.h"
@@ -33,10 +32,10 @@ std::set<Cathode> lvAliasToCathode(std::string_view alias)
3332
}
3433
}
3534

36-
std::set<int> lvAliasToDsIndices(std::string_view alias)
35+
std::set<DsIndex> lvAliasToDsIndices(std::string_view alias)
3736
{
3837
auto cathodes = lvAliasToCathode(alias);
39-
std::set<int> dsIndices;
38+
std::set<DsIndex> dsIndices;
4039
for (auto cathode : cathodes) {
4140
auto deId = cathode.deId;
4241
auto plane = cathode.plane;

Detectors/MUON/MCH/GlobalMapping/src/LV.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,11 @@
1515
#include <set>
1616
#include <string>
1717

18+
#include "MCHGlobalMapping/DsIndex.h"
19+
1820
namespace o2::mch::dcs
1921
{
20-
std::set<int> lvAliasToDsIndices(std::string_view alias);
22+
std::set<DsIndex> lvAliasToDsIndices(std::string_view alias);
2123
}
2224

2325
#endif

Detectors/MUON/MCH/GlobalMapping/src/Mapper.cxx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
namespace o2::mch::dcs
2929
{
3030

31-
std::set<int> solarAliasToDsIndices(std::string_view alias)
31+
std::set<DsIndex> solarAliasToDsIndices(std::string_view alias)
3232
{
3333
const auto chamber = aliasToChamber(alias);
3434
if (dcs::isQuadrant(chamber)) {
@@ -38,7 +38,7 @@ std::set<int> solarAliasToDsIndices(std::string_view alias)
3838
}
3939
}
4040

41-
std::set<int> aliasToDsIndices(std::string_view alias)
41+
std::set<DsIndex> aliasToDsIndices(std::string_view alias)
4242
{
4343
auto m = aliasToMeasurementType(alias);
4444
switch (m) {

Detectors/MUON/MCH/GlobalMapping/src/Quadrant.cxx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ Cathode lvAliasToCathode(std::string_view alias)
9090
}
9191
}
9292

93-
std::set<int> solarAliasToDsIndices(std::string_view alias)
93+
std::set<DsIndex> solarAliasToDsIndices(std::string_view alias)
9494
{
9595
/** For St12, to get the solar alias to dual sampas relationship
9696
* we "just" have to convert the solar crate number to a LV

Detectors/MUON/MCH/GlobalMapping/src/Quadrant.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,14 @@
1313
#define O2_MCH_GLOBAL_MAPPING_QUADRANT_H
1414

1515
#include "MCHConditions/Cathode.h"
16+
#include "MCHGlobalMapping/DsIndex.h"
1617
#include <set>
1718
#include <string>
1819

1920
namespace o2::mch::dcs::quadrant
2021
{
2122
Cathode lvAliasToCathode(std::string_view alias);
22-
std::set<int> solarAliasToDsIndices(std::string_view alias);
23+
std::set<DsIndex> solarAliasToDsIndices(std::string_view alias);
2324
} // namespace o2::mch::dcs::quadrant
2425

2526
#endif

Detectors/MUON/MCH/GlobalMapping/src/Slat.cxx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
#include "MCHConditions/DetectionElement.h"
1313
#include "MCHConditions/Number.h"
1414
#include "MCHConditions/SolarCrate.h"
15-
#include "MCHGlobalMapping/DsIndex.h"
1615
#include "MCHRawElecMap/Mapper.h"
1716
#include "Slat.h"
1817
#include <vector>
@@ -74,7 +73,7 @@ std::set<dcs::Cathode> lvAliasToCathode(std::string_view alias)
7473
return cathodes;
7574
}
7675

77-
std::set<int> solarAliasToDsIndices(std::string_view alias)
76+
std::set<DsIndex> solarAliasToDsIndices(std::string_view alias)
7877
{
7978
/** For St345 the relation solar alias to dual sampas
8079
* is a bit more involved than for quadrants.
@@ -85,7 +84,7 @@ std::set<int> solarAliasToDsIndices(std::string_view alias)
8584
*/
8685
int solarCrate = aliasToSolarCrate(alias);
8786
static auto solarIds = raw::getSolarUIDs<raw::ElectronicMapperGenerated>();
88-
std::set<int> dsIndices;
87+
std::set<DsIndex> dsIndices;
8988
for (auto solarId : solarIds) {
9089
if (solarId / 8 == solarCrate) {
9190
auto dsDetIds = raw::getDualSampas<raw::ElectronicMapperGenerated>(solarId);

Detectors/MUON/MCH/GlobalMapping/src/Slat.h

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,15 @@
1212
#ifndef O2_MCH_GLOBAL_MAPPING_SLAT_H
1313
#define O2_MCH_GLOBAL_MAPPING_SLAT_H
1414

15-
#include <set>
1615
#include "MCHConditions/Cathode.h"
16+
#include "MCHGlobalMapping/DsIndex.h"
17+
#include <set>
1718
#include <string>
1819

1920
namespace o2::mch::dcs::slat
2021
{
2122
std::set<dcs::Cathode> lvAliasToCathode(std::string_view alias);
22-
std::set<int> solarAliasToDsIndices(std::string_view alias);
23+
std::set<DsIndex> solarAliasToDsIndices(std::string_view alias);
2324
} // namespace o2::mch::dcs::slat
2425

2526
#endif

0 commit comments

Comments
 (0)