update ID ART references to rel 25.0.26

References updated to 2025-03-02T2101 nightly, used to build 25.0.26 (2025-03-03T2101 and 2025-03-04T2101 for files missing in first nightly)

@esampson

I'm not sure if it should be targeting main or r24.0 branch? Perhaps @tstreble can confirm Otherwise, I think it looks okay

resolved all threads

added 1 commit

• bfccd47f - update simreco

Compare with previous version

resolved all threads

marked this merge request as ready

This merge request affects 1 package:

• InnerDetector/InDetValidation/InDetPhysValMonitoring

This merge request affects 27 files:

• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_data15_13TeV_1000evt.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_data16_13TeV_1000evt.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_data17_13TeV_1000evt.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_data18_13TeV_1000evt.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_data22_13p6TeV_1000evt.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_data23_13p6TeV_1000evt.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_data24_13p6TeV_1000evt.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_ele10GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_ele10GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_ele5GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_ele5GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu100GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu100GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu10GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu10GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu1GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu1GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu20GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu20GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu5GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_mu5GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_pi1GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_pi1GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_pi5GeV_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_pi5GeV_simreco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_ttbarPU40_reco.sh
• InnerDetector/InDetValidation/InDetPhysValMonitoring/test/test_zprime_tide.sh

Adding @sroe ,@jburzyns ,@goetz ,@keli ,@lgagnon as watchers

added InnerDetector PhysValidation main review-pending-level-1 labels

CI Result SUCCESS (hash bfccd47f)

	Athena
externals
cmake
make
tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output (remote access info)

changed the description

Approving from L1 side (Fábio).

removed review-pending-level-1 label

added review-approved label

merged

mentioned in commit 27b1bbb8

It looks like some of the reference plots which have been updated didn't include the full statistics.

See for instance

https://atlas-art-data.web.cern.ch/atlas-art-data/grid-output/main/Athena/x86_64-el9-gcc13-opt/2025-03-09T2101/InDetPhysValMonitoring/test_ele10GeV_reco/dcube_shifter?rid=r_7

which has only one tenth of the tracks wrt the test file.

In addition all of the HitValid and RDOAnalysis references for the simreco tests seem to have gone out of sync, while the day before, no discrepancy was reported for those

https://bigpanda.cern.ch/art/jobs/?package=InDetPhysValMonitoring&branch=main/Athena/x86_64-el9-gcc13-opt&ntag_full=2025-03-06T2101

https://bigpanda.cern.ch/art/jobs/?package=InDetPhysValMonitoring&branch=main/Athena/x86_64-el9-gcc13-opt&ntag_full=2025-03-06T2101

The reco outputs also display some discrepancies

https://atlas-art-data.web.cern.ch/atlas-art-data/grid-output/main/Athena/x86_64-el9-gcc13-opt/2025-03-09T2101/InDetPhysValMonitoring/test_mu100GeV_simreco/dcube_shifter?rid=r_15

@piazza can you confirm you retrieved the updated references from the main nightly and not a r24 one?

Hi @tstreble, thanks for pointing this out!

I did take the references from r24... I should have fixed it now. I'll cross-check with the next nightly results

Excellent, thanks a lot for the prompt action

Hi @tstreble . Unfortunately there seems to be the same problem.. I'm investigating it better

@tstreble

Ok, the HitValid and RDOAnalysis issues look mostly resolved, but the low stat problem is still there.

I think it's because, in cases like this https://atlas-art-data.web.cern.ch/atlas-art-data/grid-output/main/Athena/x86_64-el9-gcc13-opt/2025-03-04T2101/InDetPhysValMonitoring/test_ele10GeV_reco/ I was taking as reference histograms the art_core_0/physval.ntuple.root files, while I should probably use the physval.root ones?

I've just done this replacement, but please let me know if it's not the correct procedure

That's correct indeed. When you have some art_core0 directories, that means that the jobs has run on several cores in parallel so the final outputs need to be merged, which is what is done in the global physval.root. This is what's implemented here

https://gitlab.cern.ch/atlas/athena/-/blob/main/InnerDetector/InDetValidation/InDetPhysValMonitoring/share/test_MC_mu0_reco.sh#L33