Reduce (temporary) memory usage in PrepareMuonHits
instead of copying (pointers to) coords into an array indexed by station, followed by a conversion (by station) into MuonHits, directly convert into MuonHits.
Also, make constants constexpr, and put them in an anonymous namespace.
Finally, add MuonCoordsToHits, which is identical to PrepareMuonHits, except the input argument is std::vectorLHCb::MuonCoord instead of LHCb::MuonCoords (which is a KeyedContainer). This should be used in conjunction with LHCb!1646 (merged) which changes the output of MuonRawToCoord to std::vectorLHCb::MuonCoord.
Edited by Marco Cattaneo