Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test t_AlignDet_AlephTPC_load failing in dev4 #1311

Closed
1 task done
andresailer opened this issue Aug 12, 2024 · 4 comments
Closed
1 task done

Test t_AlignDet_AlephTPC_load failing in dev4 #1311

andresailer opened this issue Aug 12, 2024 · 4 comments

Comments

@andresailer
Copy link
Member

Check duplicate issues.

  • Checked for duplicates

Goal

The test should pass
It fails because under the circumstances described below
There is the line

12: Info in <TGeoManager::CloseGeometry>: 189 nodes/ 24 volume UID's in Detector Geometry

But the test wants, one more node and volume

190 nodes/ 25 volume UID's in Detector Geometry

Operating System and Version

el9

compiler

gcc13

ROOT Version

6-32-patches

DD4hep Version

master

Reproducer

The test fails when building the examples separately from the rest of DD4hep.
But it passes when building DD4hep with DD4HEP_BUILD_EXAMPLES.

source /cvmfs/sft.cern.ch/lcg/views/dev4/latest/x86_64-el9-gcc13-opt/setup.sh
git clone https://github.com/aidasoft/dd4hep.git
cd dd4hep
mkdir build
cd build
cmake -GNinja \
              -DDD4HEP_USE_GEANT4=ON \
              -DBoost_NO_BOOST_CMAKE=ON \
              -DDD4HEP_USE_LCIO=ON \
              -DDD4HEP_USE_EDM4HEP=OFF \
              -DDD4HEP_USE_TBB=ON \
              -DDD4HEP_USE_HEPMC3=ON \
              -DDD4HEP_BUILD_DEBUG=OFF \
              -DBUILD_TESTING=ON \
              -DDD4HEP_DEBUG_CMAKE=ON \
              -DDD4HEP_USE_XERCESC=ON \
              -DCMAKE_BUILD_TYPE=Release \
              -DCMAKE_CXX_FLAGS="-fdiagnostics-color=always" \
              -DCMAKE_CXX_STANDARD=20 ..
ninja install
source ../bin/thisdd4hep.sh
cd ../examples
mkdir build
cd build
cmake -GNinja \
            -DBoost_NO_BOOST_CMAKE=ON \
            -DDD4HEP_USE_XERCESC=ON \
            -DCMAKE_CXX_STANDARD=20 ..
ninja
ctest -V -R t_AlignDet_AlephTPC_load

Additional context

No response

@andresailer andresailer added bug and removed bug labels Aug 12, 2024
@andresailer andresailer pinned this issue Aug 12, 2024
@andresailer
Copy link
Member Author

The problem seems to be that the libAlignDetExample.so library is loaded from DD4hep on cvmfs, and not from the examples we build ourselves.

I should probably fix the testing of DD4hep in the LCG stacks, so that it doesn't install the example libraries.

@MarkusFrankATcernch
Copy link
Contributor

MarkusFrankATcernch commented Aug 13, 2024

I think so. If I remember correctly this test had to be changed after the fixes I introduced for Ben:
#1287
Hence the additional volume actually is an assembly.

@andresailer
Copy link
Member Author

Thanks! That explains that, dev4 has DD4hep 01-29.

@andresailer
Copy link
Member Author

dev4 test is passing again.

Changed the way tests are done in the LCG nightlies. Examples are build separately and their libraries or executables are no longer deployed to cvmfs.

@andresailer andresailer unpinned this issue Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants