Skip to content

MuonHoughPatternTool - Minor tidy up

Johannes Junggeburth requested to merge jojungge/athena:HoughSimplify into master

Hi,

the actual changes introduced in this MR can be found in the second commit. The first one is just an unleashing of the clang-formater. The filling of

        std::vector<double> radius;
        std::vector<double> errradius;
        std::vector<double> weights;
        std::vector<double> prob;  // artificial probability that hit belongs to true muon
        std::vector<int> multilayer;
        std::vector<int> tubelayer;
        std::vector<int> tubes;
        std::vector<int> onsegment;  // non-zero if on segment, int indicates how many hits in same
                                     // layer are on segment (used in weighting)
        std::vector<double> psi;
        std::vector<double> weight_trigger;
        std::vector<int> tr_confirmation;  // 1 if confirmation from trigger hits else 0
        std::vector<Identifier> ids;
        std::vector<int> layers;
        std::vector<const Muon::MdtPrepData*> prd;

is abandoned in favour of filling a vector of the so-called SegmentData struct. That should simplify the memory layout here. The second change is caching the translation of station names (e.g. BMS->BML, etc. into a map of integers).

Edited by Johannes Junggeburth

Merge request reports