Draft: include muonIDHlt2 in the hlt2 dataflow

Depends on Rec!2292 (merged)

changed the description

mentioned in merge request Rec!2292 (merged)

Thanks for the MR @rvazquez .

Should we remove the protections to disable the MuonID in multi-threaded tests like for example here in this MR or in the next?

assigned to @acasaisv

I run the multithreaded tests with the new MuonIDHlt2Alg and they passed, so I guess can be disabled everywhere already.

If you can do it, it would be better as I don't know where these protections lie.

Will you push changes to these MR ? If that's the case I will wait to launch the ci-test.

Yeah, lets wip

marked as a Work In Progress

added 55 commits

• 3701f5e8...813e134a - 53 commits from branch master
• 6442e3fe - include muonIDHlt2 in the hlt2 dataflow
• 1ef4bf3b - Remove disabling of muon id

Compare with previous version

unmarked as a Work In Progress

added needs ref update label

added 1 commit

• 207959cb - Fixed formatting

Compare with previous version

I am quite sure the MR will clash with !678 (merged). So will need rebasing once this is merged.

mentioned in merge request !685 (merged)

mentioned in merge request !497 (closed)

mentioned in issue #231 (closed)

assigned to @lpica and unassigned @acasaisv

@sstahl @rvazquez !678 (merged) have been merged, it needs to be rebased?

assigned to @rvazquez and unassigned @lpica