TrigByteStreamTools: optimize trigbs_dumpHLTContentInBS_run3

Further optimization of trigbs_dumpHLTContentInBS_run3.py, i.e. the size_summary method (follow-up to !37507 (merged)):

• The main improvement comes from not converting rod_data() to a tuple. The return type of rod_data() is a u32slice and hence indexing (or getting its length) is fast.
• Use the new StringSerializer::deserialize method to avoid constructing std::vector via ROOT.
• Remove the lru_cache again as that does not seem to have a measurable impact anymore. Likely the cost of converting lists to tuples is offset by the gains of the caching.

This improves the runtime on the 1000 events from test_trig_data_v1Dev_writeBS_grid.py from 7.5 to 2.5 minutes.

Relates to ATR-24508.

This merge request affects 3 packages:

• HLT/Trigger/TrigTransforms/TrigTransform
• Trigger/TrigDataAccess/TrigSerializeResult
• Trigger/TrigTools/TrigByteStreamTools

Affected files list will not be printed in this case

Adding @fwinkl ,@bernius ,@wiedenma ,@rbielski ,@mark as watchers

added Trigger master review-pending-level-1 labels

Very nice improvement, thanks!

CI Result SUCCESS (hash 604d8bd2)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 47444]

MR looks good, CI succeeded. Approving.

Cheers, Martin (L1)

added review-approved label and removed review-pending-level-1 label

mentioned in commit de5b7b63

merged

added sweep:ignore label