Add Detector.yml and Simuation.yml to VP/2024-v00.00 VP/2024.Q1.2-v00.00 to fix access to VP conditions

This is an update to fix Boole#17 (closed) where the VP conditions were not being accessed in the standard Boole test. The conditions paths to Detector.yml and Simulation.yml were set in the VP/trunk, VP/2024-v01.00 and VP/2025-v00.00 in compact/components, but had not been set in VP/2024.Q1.2-v00.00. This causes the Boole code accessing the conditions to crash as the conditions path does not exist, the code is protected against the files being missing in lhcb-conditions-database but not missing from the Detector xml (is that possible?).

---

Validated by

• Core Software
• RTA
• Simulation

added Simulation VP labels

mentioned in issue Boole#17 (closed)

It is the same as what we have for UT!691. I think it is good to fix it, and it only adds some options to load or not load the conditions, without really changing the geometry. So, I think it is nice to have this included in !691 (merged). @gcorti

mentioned in merge request !691 (merged)

requested review from @jonrob

requested review from @sponce

requested review from @gcorti

marked the checklist item Simulation as completed

I would prefer for the code to be protected, but as it does not seems possible and this is just to ensure to handle the case that conditions are not there, I am fine with merging this.

marked this merge request as ready

approved this merge request

Could @jonrob @sponce give your approval if you are happy with those changes?

/ci-test --model lhcb-sim11

added ci-test-triggered label

• [2025-03-04 11:22] Validation started with lhcb-sim11-mr#288
• [2025-03-04 12:09] Validation started with lhcb-master-mr#12754

/ci-test

marked the checklist item RTA as completed

resolved all threads

added 8 commits

• fa810a5b...b0831978 - 6 commits from branch master
• c87ee380 - Added Detector.yml and Simuation.yml to VP/2024-v00.00 VP/2024.Q1.2-v00.00
• 25467961 - Remove extra emtpy lines

Compare with previous version

marked the checklist item Core Software as completed

approved this merge request

merged

mentioned in commit 959a4a3b