Port IsMuonTool to functional
Needs to be done someday, for now postponed.
On 13 Dec 2017, at 14:22, Nikita Kazeev nikita.kazeev@cern.ch wrote:
+Roel Aaij Roel.Aaij@cern.ch
Hi,
I've just spoke with Sascha. Here the summary:
Make Hlt IsMuonTool get the hits from TES and pass them to the new CommonMuonTool. IsMuonTool has to go, but it's not yet unclear what shall replace it.
On 13.12.2017 13:01, Nikita Kazeev wrote:
Hi,
We're working on porting MuID to GaudiFunctional. MuID appears in two places:
- Hlt Hlt/Hlt1Muons
- Rec Muon/MuonID
Muon/MuonID is self-contained, I've updated it to functional.
At Hlt/Hlt1Muons we have the IsMuonTool, which uses TES access through CommonMuonTool. IsMuonTool is used in Hlt lines - so, if we're to update it to an algorithm, we'll need to edit those lines, involve their authors, do lots of tests and risk bugs. And all this will be a wasted effort since the lines are likely to be changed during the LHCb upgrade.
I propose following:
- Muon/MuonID/MuonIDAlgLight and all its dependencies are updated to functional
- non-functional CommonMuonTool and IsMuonTool are retained, DEPRECATION warnings are added to them
- A separate functional IsMuonAlg is added