Fix Allen UT geometry

Migrate the fix from LHCb!4395 (merged) to the Allen project, which is more appropriate.

Test with LHCb!4395 (merged)

FYI: @bjashal @adeoyang @raaij @cagapopo @mveghel

Closes #467 (closed)

added RTA WP2 label

assigned to @jzhuo

mentioned in merge request LHCb!4395 (merged)

added RTA label

added 1 commit

• 76691637 - Fixed formatting

Compare with previous version

requested review from @raaij

changed the description

marked this merge request as ready

@jzhuo Could you please add a an efficiency test on the expected 2024 MC sample with a sequence including the UT (such as hlt1_pp_forward_then_matching_validation) ? I have recently added this one to test the PV performance on a sample with the beamline not placed at (0,0). It runs on a minimum bias sample however, for a tracking efficiencies BsPhiPhi would be better suited. @ascarabo has already put some in the /scratch/allen_data area in online which can be used. In addition, a throughput test would be good as well. You can just add another sequence here.

added 1 commit

• 36f535ba - Apply Aaij and Dorothea's suggestions

Compare with previous version

added 1 commit

• 1c311922 - Fixed formatting

Compare with previous version

mentioned in issue #467 (closed)

changed the description

mentioned in issue Moore#699 (closed)

@spradlin could you launch a ci test here (with LHCb!4395 (merged))?

added 1 commit

• 03770d39 - Update references

Compare with previous version

mentioned in merge request !1384 (closed)

/ci-test LHCb!4395 (merged)

mentioned in issue Moore#700 (closed)

added 45 commits

• 03770d39...d5a1e3f8 - 44 commits from branch master
• 8b836375 - Merge branch 'master' into jzhuo_FixAllenUT

Compare with previous version

mentioned in issue #496

/ci-test LHCb!4395 (merged)

added ci-test-triggered label

• [2024-02-07 11:02] Validation started with lhcb-master-mr#10432
• [2024-02-08 15:26] Validation started with lhcb-master-mr#10464
• [2024-02-22 15:27] Validation started with lhcb-master-mr#10697

added 1 commit

• 4c784260 - Fix the ci-test

Compare with previous version

added 1 commit

• 441f2b2d - Update CI references

Compare with previous version

added 1 commit

• 4a8cd887 - Fixed formatting

Compare with previous version

/ci-test LHCb!4395 (merged)

added impacthigh priorityhigh labels

added 1 commit

• f9e25ef3 - Update CI references

Compare with previous version

resolved all threads

@jzhuo there are merge conflicts, it looks like they are only in the references, where some names have changed. Could you please take care of these? Then this can hopefully be merged.

added 17 commits

• f9e25ef3...b709ad21 - 16 commits from branch master
• 831dd21e - Merge branch 'master' into jzhuo_FixAllenUT

Compare with previous version

added hlt1-throughput-decreased label

/ci-test LHCb!4395 (merged)

mentioned in issue Moore#708 (closed)

assigned to @jhorswil

added 27 commits

• 831dd21e...5f68d340 - 18 commits from branch master
• b26b03de - Fix the UT geometry in HLT1
• 31b57fa4 - Fixed formatting
• 13ba1ab9 - Apply Aaij and Dorothea's suggestions
• c217e0a1 - Fixed formatting
• 1b65cc78 - Update references
• 23df0d44 - Fix the ci-test
• 0aa098da - Update CI references
• a9c45fc1 - Fixed formatting
• f2428e6a - Update CI references

Compare with previous version

removed hlt1-throughput-decreased label

added hlt1-throughput-decreased label

added 1 commit

• d46fa518 - Update CI references

Compare with previous version

removed hlt1-throughput-decreased label

added needs Allen ref update label

resolved all threads

approved this merge request

merged

mentioned in commit 2f67f8f0

mentioned in merge request !1431 (merged)