T0 introduced to AT converter

T0 available from Analysis Tree in: -SimEventHeader -RecEventHeader +T0 subtracted from Tof time to calculate m2 correctly

@f.kornas,

could you please properly format the code. Instructions can be found at

https://redmine.cbm.gsi.de/projects/cbmroot/wiki/Clang-format

@f.uhlig

Thanks, but i cannot install the missing library using sudo -k apt install libtinfo5 as this does not work neither on lxpool nor on the lxbk maschines. Is there a way around to get the clang-format working?

@f.kornas,

thanks for informing me about the problem. We had an installation for Debian8 but not for Debian10 which I have now installed. It was simply enough to add the missing library to the lib directory. I will add the files as well as instructions for Debian10 to the Redmine Wiki.

Please find the installed version of clang-format at

/cvmfs/cbm.gsi.de/debian10/clang-format-11.0.0/bin/clang-format

@f.kornas,

I have added the file and updated the instruction at

https://redmine.cbm.gsi.de/projects/cbmroot/wiki/Clang-format

Could you please check if the instructions are understandable and work for you.

added 2 commits

• 74781f9a - T0 MC description changed
• 44a9e2ff - Fixed Formating

Compare with previous version

@f.uhlig

Perfect, now it worked, branch has been updated.

@f.kornas,

all CI jobs fail since on of the test fails.

analysis_tree_maker_sis100_electron

You can find the actual error at or CDash webpage

by choosing the failed test at

https://cdash.gsi.de/index.php?project=CbmRoot&date=2023-02-13

e.g.

https://cdash.gsi.de/testDetails.php?test=13438366&build=370473

added 2 commits

• 7d5b845c - Check cbm_header init before using it
• 9eb28ac0 - Problem fixed for m2 calculation in AT, if no CbmEvent is found

Compare with previous version

The problem with the CI test is fixed, yet i'll want to check one more thing before the merge.

added 27 commits

• 9eb28ac0...0cc3112e - 22 commits from branch computing:master
• 5568749d - T0 introduced to AT converter
• 1f01c58c - T0 MC description changed
• 61d421ac - Fixed Formating
• 43b0dacc - Check cbm_header init before using it
• c9719e1b - Problem fixed for m2 calculation in AT, if no CbmEvent is found

Compare with previous version

@f.kornas,

okay I will wait for your GO. Should this MR be added to the release candidate branch?

Dear @i.selyuzhenkov,

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.

added CodeOwners label

assigned to @f.uhlig

added 1 commit

• 8260ea21 - Added runtime error for MCEventList

Compare with previous version

I added a runtime error in CbmSimEventHeader.cxx for MCEventList such that it stops when the object isn't found.

@f.uhlig From my side, everything for T0 is in. Would be good, to have this MR included to FEB23 release.

added 1 commit

• 25025ebc - Code format fixed

Compare with previous version

changed milestone to %FEB23

approved this merge request

added 8 commits

• 215c20f6 - 1 commit from branch computing:master
• 11faab24 - T0 introduced to AT converter
• fc5c1b47 - T0 MC description changed
• 5be3f09e - Fixed Formating
• 43191df3 - Check cbm_header init before using it
• db586310 - Problem fixed for m2 calculation in AT, if no CbmEvent is found
• 981b10ae - Added runtime error for MCEventList
• 985e3cf9 - Code format fixed

Compare with previous version