Update IDEA compact file, add test of full IDEA detector #365

atolosadelgado · 2024-08-06T16:19:35Z

BEGINRELEASENOTES

IDEA_o1_v03: Dual Readout Calorimeter (DRC) is not loaded by default
- Added Test for IDEA with DRC

ENDRELEASENOTES

Hi,

I have added a test with full IDEA, this is the time it took in my laptop:

Geant4Kernel     INFO  ++ Terminate Geant4 and delete associated actions.
DDSim            INFO Total Time:   405.74 s (User), 1.58 s (System)
DDSim            INFO StartUp Time: 186.54 s, Event Processing: 219.20 s (219.20 s/Event)

test/CMakeLists.txt

atolosadelgado · 2024-08-07T07:30:20Z

Hi,

I have an issue with this test, and I do not fully understand what is going on... any help would be appreciated :)

If I compile and install k4geo,

git clone -b update_IDEA_ALLEGRO https://github.com/atolosadelgado/k4geo.git
source /cvmfs/sw-nightlies.hsf.org/key4hep/setup.sh
cmake -B build -S . -D CMAKE_INSTALL_PREFIX=install
cmake --build build -j 4 -- install

and then move to the build directory and execute the new test as follows,

cd build/
ctest -VV -R t_test_IDEA_with_DRC_o1_v03

It fails. But if I execute the command inside that test, in the same directory,- it works

../install/bin/run_test_k4geo.sh ddsim --compactFile=/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/test/compact/IDEA_withDRC_o1_v03.xml --steeringFile=/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/test/../example/SteeringFile_IDEA_o1_v03.py --gun.direction '(3,4,0)'

Any idea why a command works in the terminal, but not within ctest? (I am using Alma9)

andresailer · 2024-08-07T07:37:02Z

How does it fail when you run ctest?

test/CMakeLists.txt

atolosadelgado · 2024-08-07T07:40:47Z

How does it fail when you run ctest?

This is what is printed in the terminal when running the test:

16: Info in <TGeoManager::CheckGeometry>: Fixing runtime shapes...
16: Info in <TGeoManager::CheckGeometry>: ...Nothing to fix
16: Info in <TGeoManager::CloseGeometry>: Counting nodes...
16: Info in <TGeoManager::Voxelize>: Voxelizing...
16: Info in <TGeoManager::CloseGeometry>: Building cache...
16: Info in <TGeoManager::CountLevels>: max level = 7, max placements = 21600
16: Info in <TGeoManager::CloseGeometry>: 260170004 nodes/ 21750 volume UID's in Detector Geometry
16: Info in <TGeoManager::CloseGeometry>: ----------------modeler ready----------------
16: Compact          INFO  ++ Converted field: Successfully created field CompensatingSolenoidZminus [solenoid]
16: Detector         INFO  +++ Patching names of anonymous shapes....
16: DD4hepVolumeManager INFO  **** running plugin DD4hepVolumeManager ! 
16: VolumeManager    INFO   - populating volume ids - be patient ...
16: VolumeManager    INFO   - populating volume ids - done. 95600 nodes.
16: VolumeManager    INFO  +++ Volume manager populated and loaded.
16: InstallSurfaceManager INFO  **** running plugin InstallSurfaceManager ! 
16: InstallSurfaceManager INFO  --------  SurfaceManager contains the following maps : --------- 
16:   key: BeBeampipe 	 number of surfaces : 8
16:   key: BeamPipe 	 number of surfaces : 5
16:   key: BeamPipeShield 	 number of surfaces : 0
16:   key: BeamPipeShield_noRot 	 number of surfaces : 0
16:   key: CompSol 	 number of surfaces : 0
16:   key: DCH_v2 	 number of surfaces : 0
16:   key: DRcalo 	 number of surfaces : 0
16:   key: EndPlateAbsorber_negative 	 number of surfaces : 0
16:   key: EndPlateAbsorber_positive 	 number of surfaces : 0
16:   key: LumiCal 	 number of surfaces : 0
16:   key: LumiCalBackShield 	 number of surfaces : 0
16:   key: LumiCalCooling 	 number of surfaces : 0
16:   key: LumiCalInstrumentation 	 number of surfaces : 0
16:   key: Muon-System 	 number of surfaces : 0
16:   key: ScreenSol 	 number of surfaces : 0
16:   key: Solenoid 	 number of surfaces : 0
16:   key: SolenoidBarrel 	 number of surfaces : 0
16:   key: SolenoidEndcaps 	 number of surfaces : 0
16:   key: SynchRadMask 	 number of surfaces : 0
16:   key: VTXD 	 number of surfaces : 5408
16:   key: VTXIB 	 number of surfaces : 906
16:   key: VTXOB 	 number of surfaces : 4000
16:   key: Vertex 	 number of surfaces : 10314
16:   key: compound 	 number of surfaces : 10314
16:   key: passive 	 number of surfaces : 13
16:   key: tracker 	 number of surfaces : 10314
16:   key: world 	 number of surfaces : 20641
16: ---------------------------------------------------------------- 
16: 
16: DDG4             INFO +++ Imported 333 global values to namespace:DDG4
16: Geant4Kernel           OutputLevel:  2
16: Geant4Kernel           UI:           UI
16: Geant4Kernel           NumEvents:    10
16: DDG4             INFO +++  List of sensitive detectors:
16: Geant4Kernel           NumThreads:   0
16: DDG4             INFO +++  DCH_v2                           type:tracker       --> Sensitive type: ('Geant4TrackerWeightedAction', {'HitPositionCombination': 2, 'CollectSingleDeposits': False})
16: DDG4             INFO +++  DRcalo                           type:DRcaloSiPMSD  --> Sensitive type: Unknown
16: DDG4             INFO +++  LumiCal                          type:calorimeter   --> Sensitive type: Geant4ScintillatorCalorimeterAction
16: DDG4             INFO +++  LumiCalBackShield                type:calorimeter   --> Sensitive type: Geant4ScintillatorCalorimeterAction
16: DDG4             INFO +++  LumiCalCooling                   type:calorimeter   --> Sensitive type: Geant4ScintillatorCalorimeterAction
16: DDG4             INFO +++  LumiCalInstrumentation           type:calorimeter   --> Sensitive type: Geant4ScintillatorCalorimeterAction
16: DDG4             INFO +++  Muon-System                      type:tracker       --> Sensitive type: ('Geant4TrackerWeightedAction', {'HitPositionCombination': 2, 'CollectSingleDeposits': False})
16: DDG4             INFO +++  VTXD                             type:tracker       --> Sensitive type: ('Geant4TrackerWeightedAction', {'HitPositionCombination': 2, 'CollectSingleDeposits': False})
16: DDG4             INFO +++  VTXIB                            type:tracker       --> Sensitive type: ('Geant4TrackerWeightedAction', {'HitPositionCombination': 2, 'CollectSingleDeposits': False})
16: DDG4             INFO +++  VTXOB                            type:tracker       --> Sensitive type: ('Geant4TrackerWeightedAction', {'HitPositionCombination': 2, 'CollectSingleDeposits': False})
16: Geant4UI         INFO  +++ UI> Install Geant4 control directory:/ddg4/UI/
16: Geant4Kernel     INFO  ++ Registered global action UI of type dd4hep::sim::Geant4UIManager
16: Geant4UI         INFO  +++ MagFieldTrackingSetup> Install Geant4 control directory:/ddg4/MagFieldTrackingSetup/
16: Geant4UI         INFO  +++ DetectorConstructionAction> Install Geant4 control directory:/ddg4/DetectorConstructionAction/
16: Geant4UI         INFO  +++ ConstructSD> Install Geant4 control directory:/ddg4/ConstructSD/
16: Geant4Kernel     INFO  ++ Registered global action RunInit of type dd4hep::sim::Test::Geant4TestRunAction
16: DDSim.Helper.Random INFO Choosing random seed for you: 1114215023
16: DDSim.Helper.OutputConfig INFO ++++ Setting up UserPlugin for Output ++++
16: Geant4UI         INFO  +++ RunAction> Install Geant4 control directory:/ddg4/RunAction/
16: testIDEA_o1_v03.root INFO  Writer is now instantiated ...
16: Geant4UI         INFO  +++ testIDEA_o1_v03.root> Install Geant4 control directory:/ddg4/testIDEA_o1_v03.root/
1/1 Test #16: t_test_IDEA_with_DRC_o1_v03 ......***Exception: SegFault 80.44 sec

0% tests passed, 1 tests failed out of 1

Total Test time (real) =  80.44 sec

The following tests FAILED:
	 16 - t_test_IDEA_with_DRC_o1_v03 (SEGFAULT)
Errors while running CTest
Output from these tests are in: /home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.

This is the log file:

tail -n 20  /home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build/Testing/Temporary/LastTest.log
Geant4UI         INFO  +++ UI> Install Geant4 control directory:/ddg4/UI/
Geant4Kernel     INFO  ++ Registered global action UI of type dd4hep::sim::Geant4UIManager
Geant4UI         INFO  +++ MagFieldTrackingSetup> Install Geant4 control directory:/ddg4/MagFieldTrackingSetup/
Geant4UI         INFO  +++ DetectorConstructionAction> Install Geant4 control directory:/ddg4/DetectorConstructionAction/
Geant4UI         INFO  +++ ConstructSD> Install Geant4 control directory:/ddg4/ConstructSD/
Geant4Kernel     INFO  ++ Registered global action RunInit of type dd4hep::sim::Test::Geant4TestRunAction
DDSim.Helper.Random INFO Choosing random seed for you: 1114215023
DDSim.Helper.OutputConfig INFO ++++ Setting up UserPlugin for Output ++++
Geant4UI         INFO  +++ RunAction> Install Geant4 control directory:/ddg4/RunAction/
testIDEA_o1_v03.root INFO  Writer is now instantiated ...
Geant4UI         INFO  +++ testIDEA_o1_v03.root> Install Geant4 control directory:/ddg4/testIDEA_o1_v03.root/
<end of output>
Test time =  80.44 sec
----------------------------------------------------------
Test Failed.
"t_test_IDEA_with_DRC_o1_v03" end time: Aug 07 09:34 CEST
"t_test_IDEA_with_DRC_o1_v03" time elapsed: 00:01:20
----------------------------------------------------------

End testing: Aug 07 09:34 CEST

andresailer · 2024-08-07T07:43:43Z

Doesn't ctest also print something about the command it runs and the environment?

atolosadelgado · 2024-08-07T07:45:39Z

Doesn't ctest also print something about the command it runs and the environment?

Yes:

ctest -VV -R t_test_IDEA_with_DRC_o1_v03
UpdateCTestConfiguration  from :/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build/DartConfiguration.tcl
Parse Config file:/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build/DartConfiguration.tcl
UpdateCTestConfiguration  from :/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build/DartConfiguration.tcl
Parse Config file:/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build/DartConfiguration.tcl
Test project /home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build
Constructing a list of tests
Done constructing a list of tests
Updating test list for fixtures
Added 0 tests to meet fixture requirements
Checking test dependency graph...
Checking test dependency graph end
test 16
    Start 16: t_test_IDEA_with_DRC_o1_v03

16: Test command: /home/alvarotd/work/update_main_detectors_after_DRC/k4geo/install/bin/run_test_k4geo.sh "ddsim --compactFile=/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/test/compact/IDEA_withDRC_o1_v03.xml --steeringFile=/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/test/../example/SteeringFile_IDEA_o1_v03.py --gun.direction '(3,4,0)'
 "
16: Working Directory: /home/alvarotd/work/update_main_detectors_after_DRC/k4geo/build/test
16: Test timeout computed to be: 1200
16: ./thisroot.sh: line 211: warning: command substitution: ignored null byte in input
16:  #### LD_LIBRARY_PATH = :  /home/alvarotd/work/update_main_detectors_after_DRC/k4geo/install/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/dd4hep/9f142b8d4caad59970244e593cac170bf5d262c4_develop-cjnxpg/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/geant4/11.2.2-52lagt/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/root/6.32.02-bwmxwc/lib/root:/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/install/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/libtool/2.4.7-txx4da/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/fastjet/3.4.2-6pryjc/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/vdt/0.4.4-n3iir7/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/opendatadetector/8392fda4b61df80848ff45745f53c05703081d8c_develop-gmtxf2/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/opendatadetector/8392fda4b61df80848ff45745f53c05703081d8c_develop-gmtxf2/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4rectracker/bf6415860a98b59a4b470ef9cdf8927de187601e_develop-t3yzpy/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4rectracker/bf6415860a98b59a4b470ef9cdf8927de187601e_develop-t3yzpy/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4reco/2a75b5a90fdeac1249809e8380481fa61629b478_develop-a4rsle/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4reco/2a75b5a90fdeac1249809e8380481fa61629b478_develop-a4rsle/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4clue/278e94ef763111ed83993de9f3b4557e906b5069_develop-7sazlu/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4clue/278e94ef763111ed83993de9f3b4557e906b5069_develop-7sazlu/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4marlinwrapper/fef47b251bb1b048330eda098d495c618d8377f5_develop-rfbkce/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4marlinwrapper/fef47b251bb1b048330eda098d495c618d8377f5_develop-rfbkce/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/intel-oneapi-mkl/2024.2.0-sul5pr/mkl/2024.2/lib::/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/gosam-contrib/2.0-aaan2s/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/gsl/2.7.1-5fdd2b/lib/root:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/root/6.32.02-bwmxwc/lib/root:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4simdelphes/46af66c42e7dd5b913665dcbc93fde6bc6118b6f_develop-6sbper/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4simdelphes/46af66c42e7dd5b913665dcbc93fde6bc6118b6f_develop-6sbper/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4reccalorimeter/c954960351e3d654ba2e1c5aa963eccd35ba5b9c_develop-ptqkbf/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4reccalorimeter/c954960351e3d654ba2e1c5aa963eccd35ba5b9c_develop-ptqkbf/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4simgeant4/1a66dc9960223b9f14e18cea23965144f3e3cd86_develop-hrmsic/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4simgeant4/1a66dc9960223b9f14e18cea23965144f3e3cd86_develop-hrmsic/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/fccdetectors/9cad5e2117982541d2cebcb360c4577320c47ca2_develop-2kbeaq/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/fccdetectors/9cad5e2117982541d2cebcb360c4577320c47ca2_develop-2kbeaq/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/lcio/34fd6f89c3215e8682e71ab27eae74769a4fb325_develop-tyao56/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/py-onnxruntime/1.17.1-l7j2tc/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/py-onnxruntime/1.17.1-l7j2tc/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/fccanalyses/dae0778e138b87f5f8400c98a931d85608c24385_develop-7cqckp/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/fccanalyses/dae0778e138b87f5f8400c98a931d85608c24385_develop-7cqckp/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4gen/b09f294cceda628e7c6d32e2f1d41857f8ec6a05_develop-bphbh7/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4gen/b09f294cceda628e7c6d32e2f1d41857f8ec6a05_develop-bphbh7/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4fwcore/a03713c2b6e27de008c352ee64dba84eef38ba65_develop-ye2wg3/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/k4fwcore/a03713c2b6e27de008c352ee64dba84eef38ba65_develop-ye2wg3/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/gaudi/38.3-4fl56o/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/gaudi/38.3-4fl56o/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/edm4hep/a173bdec749abf5447810b25286a64456914faa6_develop-wtqsc2/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/sio/9368e91bb3f4a99e8286d26dc7a425f216ff97d7_develop-vqzlp4/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/podio/1dd23dfcde95c5fbf9317db1b8aa8035dfb57b83_develop-j2ijhr/lib64:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/r/4.4.0-fsuqv5/rlib/R/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/hdf5/1.14.3-cgiwee/lib:::/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/clhep/2.4.7.1-pwb7pn/lib:/cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/xerces-c/3.2.5-ylma33/lib
16:  #### DYLD_LIBRARY_PATH = :  /cvmfs/sw-nightlies.hsf.org/key4hep/releases/2024-07-24/x86_64-almalinux9-gcc11.4.1-opt/root/6.32.02-bwmxwc/lib/root
16:  ### running test :  'ddsim --compactFile=/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/test/compact/IDEA_withDRC_o1_v03.xml --steeringFile=/home/alvarotd/work/update_main_detectors_after_DRC/k4geo/test/../example/SteeringFile_IDEA_o1_v03.py --gun.direction '(3,4,0)'
16:   '

BrieucF · 2024-08-07T14:52:59Z

Hi all,

The test test_IDEA_o1_v03 is failing because it calls a steering file which requires FastSimOpFiberRegion to be defined, but since the dual readout was removed it is not defined anymore. Probably removing the steering file loading from

k4geo/test/CMakeLists.txt

Line 94 in d481eb0

    
           ddsim --compactFile=${CMAKE_CURRENT_SOURCE_DIR}/../FCCee/IDEA/compact/IDEA_o1_v03/IDEA_o1_v03.xml --steeringFile=${CMAKE_CURRENT_SOURCE_DIR}/../example/SteeringFile_IDEA_o1_v03.py )

will do the job

atolosadelgado · 2024-08-07T16:27:51Z

The problem in ctest is related to the use of single and double quotes together. After removing both it works.
The seed is fixed for both IDEA with and without DRC, and the direction is such that the muon crosses the barrel and generate the optical photons (you know because the many warnings thrown by the optical physics)

atolosadelgado · 2024-08-07T16:28:07Z

Hi all,

The test test_IDEA_o1_v03 is failing because it calls a steering file which requires FastSimOpFiberRegion to be defined, but since the dual readout was removed it is not defined anymore. Probably removing the steering file loading from

k4geo/test/CMakeLists.txt

Line 94 in d481eb0

ddsim --compactFile=${CMAKE_CURRENT_SOURCE_DIR}/../FCCee/IDEA/compact/IDEA_o1_v03/IDEA_o1_v03.xml --steeringFile=${CMAKE_CURRENT_SOURCE_DIR}/../example/SteeringFile_IDEA_o1_v03.py )

will do the job

thanks for the hint, you were right!

atolosadelgado · 2024-08-09T14:13:22Z

this PR will make CI test to pass in k4RecTracker repo,
key4hep/k4RecTracker#27

is there anything I can do? :)

atolosadelgado changed the title ~~Update idea allegro~~ Update IDEA compact file, add test of full IDEA detector Aug 6, 2024

andresailer reviewed Aug 7, 2024

View reviewed changes

test/CMakeLists.txt Outdated Show resolved Hide resolved

andresailer reviewed Aug 7, 2024

View reviewed changes

test/CMakeLists.txt Outdated Show resolved Hide resolved

atolosadelgado added 9 commits August 7, 2024 19:10

comment include of DRC in IDEA o1 v3

0c0fa9d

IDEA with DRC dedicated test added

dca19c6

fix IDEA with DRC test

e5d7f86

apply other SD action to DRC

201a34d

update IDEA README

862e47c

still not working...

b3e421b

replace absolute path by cmake variables in ctest

4af1055

fix test_IDEA_o1_v03 by removing DRC steering file

ce91967

IDEA+DRC test is now working

d45c442

atolosadelgado force-pushed the update_IDEA_ALLEGRO branch from 11540a9 to d45c442 Compare August 7, 2024 17:11

BrieucF approved these changes Aug 8, 2024

View reviewed changes

andresailer merged commit 9895fd5 into key4hep:main Aug 9, 2024
5 of 6 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update IDEA compact file, add test of full IDEA detector #365

Update IDEA compact file, add test of full IDEA detector #365

atolosadelgado commented Aug 6, 2024 •

edited by andresailer

Loading

atolosadelgado commented Aug 7, 2024

andresailer commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024 •

edited

Loading

andresailer commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024

BrieucF commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024

atolosadelgado commented Aug 9, 2024

Update IDEA compact file, add test of full IDEA detector #365

Update IDEA compact file, add test of full IDEA detector #365

Conversation

atolosadelgado commented Aug 6, 2024 • edited by andresailer Loading

atolosadelgado commented Aug 7, 2024

andresailer commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024 • edited Loading

andresailer commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024

BrieucF commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024

atolosadelgado commented Aug 7, 2024

atolosadelgado commented Aug 9, 2024

atolosadelgado commented Aug 6, 2024 •

edited by andresailer

Loading

atolosadelgado commented Aug 7, 2024 •

edited

Loading