Revist necessity of std::map as container for Muon tracks in NNMuonTrack

The following discussion from !3484 (merged) should be addressed:

• @gunther started a discussion: (+16 comments)

  is there any reason to fill the pointers to muon clusters into an (ordered) map starting with an arbitrary key? why not push them into a vector?