Improve low-mass dielectron lines
- Remove "Noip" from the dielectron lines since, after splitting the line up to allow more targetted pre-scaling, some sub-lines end up having an implicit ip cut.
- Cut on track IPchi2 not vertex IPchi2 (which is just pointing-angle)
- Remove the prompt_vertex_evaluator dependence of the code to put the selection in one place
(this will need Moore!2626 (merged) to be merged at the 'same' time given the change in HLT1 line names here that that MR then filters on)
Merge request reports
Activity
assigned to @kaaricha
added RTA label
mentioned in issue Moore#558 (closed)
mentioned in issue Moore#563 (closed)
Resolved by Daniel JohnsonPlease update the formatting!
mentioned in issue Moore#574 (closed)
added 88 commits
-
2c1cc717...50dc024e - 86 commits from branch
master - 0657b1a0 - Remove Noip naming in lowmass dielectron lines
- 236eef08 - Rename the source files to remove Noip
-
2c1cc717...50dc024e - 86 commits from branch
Resolved by Fernando Luiz Ferreira Rodrigues/ci-test
- Last reply by Fernando Luiz Ferreira Rodrigues
added ci-test-triggered label
- [2023-05-25 03:15] Validation started with lhcb-master-mr#8016
- Resolved by Daniel Johnson
Seems like we now have 2 failures in the pipeline : physics-efficiency physics-efficiency-full
added 1 commit
- deba6824 - Apply IPchi2 thresholds to electron tracks not dielectron vertex
I'm a bit stuck here: my line uses secondary vertices as inputs:
dev_consolidated_svs_t=secondary_vertices["dev_consolidated_svs"], dev_sv_offsets_t=secondary_vertices["dev_sv_offsets"],and I had a neat suggestion from @kaaricha to cut on ipchi2 of the child tracks using an access like this:
const auto track1 = static_cast<const Allen::Views::Physics::BasicParticle*>(vertex.child(0));then
track1.ip_chi2()etc.
But that assumes the input to the line is, as in her case,
dev_particle_container_t=secondary_vertices[ "dev_multi_event_composites"],So:
- Should I be using multi-event-composites as input? Except that my line handles electrons, not muons as in Kate's case, I'd have thought we'd be using similar if not the same inputs?
- Is there an easier way to access the ipchi2 for a track without changing my line significantly (at this point in the dev cycle)?
Resolved by Daniel Johnsonmarking as draft until the pipeline (build) is fixed