Skip to content
Snippets Groups Projects

Update geometry hash

Merged Eoin Clerkin requested to merge e.clerkin/cbmroot:update_setups into master
All threads resolved!

Remove psd from all setups. Correct title name of setups.

Removes Psd module task from analysis tree maker when in event based mode for cases when the psd is not active.

Edited by Eoin Clerkin

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Resolved by Eoin Clerkin

      Unfortunately it seem that the analysis tree converter is expecting the PSd to be present and just crashes if it is not there.
      let's hope that this is a simple macro/json parameter tuning which could be attached to this MR. Worst case it may be something that has to be fixed in the analysis tree external repo, in which case it may take longer (I confess I have no clue how this part of the code is organized internally).

      Reference: end of the log of vae24-linux-x86_64-gcc10-fairsoft_jan24p1-fairroot_v18.8.2 job

      As we may need to have new geometries merged today for mCBM, I will maybe bring a "revert commit" in the master of the geo repository if we do not find such quick solution before they become available

      Edited by Pierre-Alain Loizeau
    • Resolved by Eoin Clerkin

      AnalysisTree fails with the following error.

      [INFO] FairRunAna::Run() After checking, the run will run from event 0 to 20. [INFO] CbmMatchRecoToMC::Exec eventNo=0 [INFO] MCEvent 1 with T0 1000 [INFO] TIME: -1 -1 [INFO] Writing MC-tracks from event # 0 file 0 root.exe: /tmp/custom-executor1174291168/builds/computing/cbmroot/analysis/common/analysis_tree_converter/CbmPsdModulesConverter.cxx:36: virtual void CbmPsdModulesConverter::ProcessData(CbmEvent*): Assertion `cbm_psd_hits_' failed.

  • Eoin Clerkin added 1 commit

    added 1 commit

    • 5eca0377 - Check if Psd is active in setup

    Compare with previous version

  • Eoin Clerkin added 1 commit

    added 1 commit

    • 82dacdd2 - Check if Psd is active in setup

    Compare with previous version

  • Eoin Clerkin added 10 commits

    added 10 commits

    Compare with previous version

  • Eoin Clerkin added 1 commit

    added 1 commit

    • 7ce50394 - Check if Psd is active in setup

    Compare with previous version

    • Resolved by Eoin Clerkin

      Error message:

      -I- FairRunTimeDB::InitContainer() CbmTrdParSetGeo [ERROR] init() CbmTrdParSetGeo not initialized -I- FairRunTimeDB::InitContainer() CbmTofDigiPar [ERROR] init() CbmTofDigiPar not initialized -I- FairRunTimeDB::InitContainer() CbmTofDigiBdfPar Container CbmTofDigiBdfPar initialized from ROOT file. Error in <FairRuntimeDb::initContainers()>: Error occured during initialization [INFO] create CbmFieldPar container CbmFieldPar Error in <FairRuntimeDb::getContainer(Text_t*)>: Container CbmGeoStsPar not created! Error in <FairRuntimeDb::getContainer(Text_t*)>: Container CbmStsDigiPar not created! ************************************************************* initialisation for run id 1741603949 ************************************************************* -I- FairRunTimeDB::InitContainer() FairGeoParSet -I- FairRunTimeDB::InitContainer() FairBaseParSet -I- FairRunTimeDB::InitContainer() CbmStsParSim -I- FairRunTimeDB::InitContainer() CbmStsParSetModule -I- FairRunTimeDB::InitContainer() CbmStsParSetSensor -I- FairRunTimeDB::InitContainer() CbmStsParSetSensorCond -I- FairRunTimeDB::InitContainer() CbmTrdParSetAsic -I- FairRunTimeDB::InitContainer() CbmTrdParSetGas -I- FairRunTimeDB::InitContainer() CbmTrdParSetDigi -I- FairRunTimeDB::InitContainer() CbmTrdParSetGain -I- FairRunTimeDB::InitContainer() CbmTrdParSetGeo [ERROR] init() CbmTrdParSetGeo not initialized -I- FairRunTimeDB::InitContainer() CbmTofDigiPar [ERROR] init() CbmTofDigiPar not initialized -I- FairRunTimeDB::InitContainer() CbmTofDigiBdfPar -I- FairRunTimeDB::InitContainer() CbmFieldPar

      .

      .

      .

      .

      .

      [INFO] ============================================================ [WARN] FairMonitor was disabled. Nothing to print! [INFO] Run: Execution successful [INFO] Run: Digitization file was data/s100c_ts.raw.root [INFO] Run: Parameter file was data/s100c_coll.par.root [INFO] Run: Output file is data/s100c_offline_ts_tb.reco.root [INFO] Run: Execution time: Init 13.1795 s, Exec 0.105718s Error in <THashList::Delete>: A list is accessing an object (0x55d8fa55a930) already deleted (list name = THashList) Error in <THashList::Delete>: A list is accessing an object (0x55d8fa656d80) already deleted (list name = THashList) Error in <THashList::Delete>: A list is accessing an object (0x55d8f6a170a0) already deleted (list name = THashList) Error in <THashList::Delete>: A list is accessing an object (0x55d8f6d797a0) already deleted (list name = THashList) [INFO] CBM Offline Reconstruction: Program completed successfully; exiting. *** Break *** segmentation violation Generating stack trace... 0x00007f5b07847463 in TList::~TList() at /usr/include/c++/12/bits/shared_ptr_base.h:1166 from /opt/fairsoft/apr21p2/lib/libCore.so.6.22 0x00007f5b078475f9 in TList::~TList() at /opt/fairsoft/source/apr21p2/build/Source/root/core/cont/src/TList.cxx:94 from /opt/fairsoft/apr21p2/lib/libCore.so.6.22 0x00007f5b07797d73 in TROOT::~TROOT() at /opt/fairsoft/source/apr21p2/build/Source/root/core/base/src/TROOT.cxx:879 (discriminator 3) from /opt/fairsoft/apr21p2/lib/libCore.so.6.22 0x00007f5b0745d55d in <unknown> from /lib/x86_64-linux-gnu/libc.so.6 0x00007f5b0745d69a in <unknown> from /lib/x86_64-linux-gnu/libc.so.6 0x00007f5b07446251 in <unknown> from /lib/x86_64-linux-gnu/libc.so.6 0x00007f5b07446305 in __libc_start_main + 0x85 from /lib/x86_64-linux-gnu/libc.so.6 0x000055d8f1b485d1 in <unknown> from /home/gitlab-runner/builds/SdZv9abz/0/computing/cbmroot/build/bin/cbmreco_offline

      Edited by Eoin Clerkin
  • Eoin Clerkin resolved all threads

    resolved all threads

  • Pierre-Alain Loizeau approved this merge request

    approved this merge request

  • Pierre-Alain Loizeau enabled an automatic merge when all merge checks for 7ce50394 pass

    enabled an automatic merge when all merge checks for 7ce50394 pass

  • Dear @f.uhlig, @a.andronic, @k.piasecki,

    you have been identified as code owner of at least one file which was changed with this merge request.

    Please check the changes and approve them or request changes.

  • Pierre-Alain Loizeau canceled the automatic merge

    canceled the automatic merge

  • Eoin Clerkin resolved all threads

    resolved all threads

  • Eoin Clerkin approved this merge request

    approved this merge request

  • Eoin Clerkin changed the description

    changed the description

Please register or sign in to reply
Loading