Fix forward no UT and add optimisation

Found issue in chi2 cut in forward noUT which affects also forward_then_matching_no_ut (no changes in sequences with UT).

Expecting ref update

We can profit from the new ghost prob NN cut to release a bit the thresholds, in practice with +1% ghost we can gain a +7% efficiency at high momentum (pt > 1 GeV and p > 5 GeV)

./toolchain/wrapper ./Allen --sequence hlt1_pp_no_ut_validation --mdf /scratch/allen_data/mdf_input/Beam6800GeV-expected-2024-MagDown-nu7.6_Bs2PhiPhiMD_nSciFiHits.mdf -g ../input/allen_geometries/geometry_dddb-20231017_sim-20231017-vc-md100_new_SciFi_geometry/

See before:

New optimisation:

FYI @dovombru @dtou

assigned to @ascarabo

changed target branch from master to 2024-patches

added RTA label

changed the description

added 1 commit

• 97302df8 - add optimised version

Compare with previous version

changed title from Fix forward no UT to Fix forward no UT and add optimisation

changed the description

changed the description

changed milestone to %RTA/2024.08.12-AugMD

approved this merge request

changed the description

added hlt1-throughput-decreased label

added 1 commit

• 6bb6f508 - Disable lines during velo closing

Compare with previous version

removed hlt1-throughput-decreased label

/ci-test

added ci-test-triggered label

• [2024-07-10 15:12] Validation started with lhcb-2024-patches-mr#996
• [2024-07-31 17:42] Validation started with lhcb-2024-patches-mr#1179
• [2024-08-01 11:48] Validation started with lhcb-2024-patches-mr#1184

@dovombru @dtou @cagapopo Any comments to this MR?

requested review from @cagapopo, @dovombru, and @dtou