Try setting the maximum number of tubes to 120 when building MDT GeoSerialIdentifiers

Hi,

as the name suggests, this MR is to find potential issues in the incremental build when setting the maximum number of tubes to 120 when building MDT GeoSerialIdentifiers.

Best, Nico

MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry/src/MdtReadoutElement.cxx

@@ -150,7 +150,7 @@ double MdtReadoutElement::getTubeLengthForCaching(int tubeLayer, int tube) const
         // usually the tube number corresponds to the child number, however for
         // BMG chambers full tubes are skipped during the building process
         // therefore the matching needs to be done via the volume ID
-        int packed_id = tube + 100*tubeLayer;
+        int packed_id = tube + 120*tubeLayer;
         int kk = 0;
         bool found = false;
         geoGetIds ([&](int id) {
@@ -162,7 +162,7 @@ double MdtReadoutElement::getTubeLengthForCaching(int tubeLayer, int tube) const
                    }, &*cv);
         if (found && log.level()<=MSG::DEBUG) {
           log << MSG::DEBUG << " MdtReadoutElement tube match found for BMG - input : tube(" << tube  << "), layer(" <<  tubeLayer 
-                    << ") - output match : tube(" << ii%100 << "), layer(" << ii/100 << ")" << endmsg;
+                    << ") - output match : tube(" << ii%120 << "), layer(" << ii/120 << ")" << endmsg;
         }
       }
       if (ii>=nGrandchildren) {
@@ -482,7 +482,7 @@ MdtReadoutElement::nodeform_localTubePos(int multilayer, int tubelayer, int tube
           // usually the tube number corresponds to the child number, however for
           // BMG chambers full tubes are skipped during the building process
           // therefore the matching needs to be done via the volume ID
-          int packed_id = tube + 100*tubelayer;
+          int packed_id = tube + 120*tubelayer;
           int kk = 0;
           bool found = false;
           geoGetIds ([&](int id) {
@@ -495,7 +495,7 @@ MdtReadoutElement::nodeform_localTubePos(int multilayer, int tubelayer, int tube
 #ifndef NDEBUG
           if (found && log.level()<=MSG::DEBUG) {
             log << MSG::DEBUG << " MdtReadoutElement tube match found for BMG - input : tube(" << tube  << "), layer(" <<  tubelayer 
-                      << ") - output match : tube(" << ii%100 << "), layer(" << ii/100 << ")" << endmsg;
+                      << ") - output match : tube(" << ii%120 << "), layer(" << ii/120 << ")" << endmsg;
           }
 #endif
         }
@@ -1753,7 +1753,7 @@ void MdtReadoutElement::fillCache()
                 // usually the tube number corresponds to the child number, however for
                 // BMG chambers full tubes are skipped during the building process
                 // therefore the matching needs to be done via the volume ID
-                int packed_id = tube + 100*tl;
+                int packed_id = tube + 120*tl;
                 bool found = false;
                 geoGetIds ([&](int id) {
                              if (!found && id == packed_id) {