Skip to content

xAOD::EventAuxInfo Schema Evolution, master branch (2020.01.30.)

After an e-mail thread with @tsulaia, @goetz and @ssnyder, we/I decided that the averageInteractionsPerCrossing and actualInteractionsPerCrossing variables of xAOD::EventInfo should become dynamic auxiliary variables. This will allow us to set them as decorations during RAW->ESD reconstruction, using a regular algorithm. (And not in the ByteStream converter.) So the only difference between xAOD::EventAuxInfo_v1 and xAOD::EventAuxInfo_v2 is that I removed those two float variables from the latter.

I tried to add and update the relevant tests. I also checked with some additional printouts in some test jobs that the pileup values seem to be read correctly into the _v2 auxiliary object correctly from old/current xAOD files. So I'm very confident that this update should "work". 😉

At the same time this is the first time that I'm using Scott's "T/P converter testing infrastructure", so he may have a comment about the code that I wrote.

BTW, Scott, you'll see that in the updated CMakeLists.txt file I search for your CMake code with the path ${CMAKE_SOURCE_DIR}/../../Database/AthenaPOOL/AthenaPoolUtilities. This is the correct path ever since we switched to Git. So I'm quite worried that many current converter packages, which use ${CMAKE_SOURCE_DIR}/Database/AthenaPOOL/AthenaPoolUtilities, are only working by chance at the moment... 😦

Merge request reports

Loading