Make pathExtract more robust - option to give either suffixes or full stream names

Fix pathExtract, used by some tools such as hotSpotInHIST and checkCorrelInHIST, to be a bit more robust when it comes to expanding the stream name. If the prefix is already present in the provided stream, don't do a lookup.

This merge request affects 1 package:

• DataQuality/DataQualityUtils

This merge request affects 3 files:

• DataQuality/DataQualityUtils/python/pathExtract.py
• DataQuality/DataQualityUtils/scripts/checkCorrelInHIST.py
• DataQuality/DataQualityUtils/scripts/hotSpotInHIST.py

added DQ main review-pending-level-1 labels

CI Result SUCCESS (hash fe386085)

	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 1
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 4705]

MR is given back to the developer due to CI test failure.

The log output as follows

# CMake ---> /cvmfs/atlas.cern.ch/repo/ATLASLocalRootBase/x86_64/Cmake/3.27.5/Linux-x86_64/bin/cmake -E touch /build2/ci-builds/main/Athena/build/Athena/DataQuality/DataQualityUtils/CMakeFiles/pyCompile.stamp
/var/lib/jenkins/workspace/CI-MERGE-REQUEST-EL9/main/DataQuality/DataQualityUtils/python/pathExtract.py:106:5: warning: F841 local variable 'prefix' is assigned to but never used
prefix = {'express': 'express_', 'Egamma': 'physics_',
^
/var/lib/jenkins/workspace/CI-MERGE-REQUEST-EL9/main/DataQuality/DataQualityUtils/python/pathExtract.py:141:5: warning: F841 local variable 'prefix' is assigned to but never used
prefix = {'express': 'express_', 'Egamma': 'physics_',
^
warning: Python code violations found. See <https://gitlab.cern.ch/atlas/atlasexternals/tree/main/External/flake8_atlas>.

http://atlas-computing.web.cern.ch/atlas-computing/links/distDirectory/ci/CIWebArea/nicos_web_areaMRCIbuilds64BE9G13AthenaOpt/NICOS_Log_MR-68910-2024-02-15-11-48/DataQuality_DataQualityUtils.html

Cheers, Yuriy (L1)

removed review-pending-level-1 label

added review-user-action-required label

added 1 commit

• 08c04984 - Remove extra definitions of prefix dict

Compare with previous version

removed review-user-action-required label

Sorry @yvolkotr I accidentally left some remnants of the old code. Should be better now

Jenkins please retry a build

added review-approved label

removed review-approved label

This merge request affects 1 package:

• DataQuality/DataQualityUtils

This merge request affects 3 files:

• DataQuality/DataQualityUtils/python/pathExtract.py
• DataQuality/DataQualityUtils/scripts/checkCorrelInHIST.py
• DataQuality/DataQualityUtils/scripts/hotSpotInHIST.py

added review-pending-level-1 label

removed review-pending-level-1 label

added review-approved label

CI Result SUCCESS (hash 08c04984)

	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 [CI-MERGE-REQUEST-EL9 4715]

merged

mentioned in commit 1377cabe