Add templated mergers
Add templated SOACollectionMerger
and VectorOfObjectsMerger
, to make merging of SOACollection-based classes and std::vector<Object>
collections easier.
Merge request reports
Activity
assigned to @decianm
Thanks, this seems useful indeed. I think we had some implementation of this but not generic (for instance https://gitlab.cern.ch/lhcb/Rec/-/blob/master/Tr/TrackUtils/src/TracksVPMerger.cpp).
- Resolved by Michel De Cian
added RTA label
- Resolved by Michel De Cian
mentioned in merge request Rec!3001 (closed)
- [2022-07-05 16:17] Validation started with lhcb-master-mr#4948
- [2022-07-25 14:17] Validation started with lhcb-master-mr#5188
Edited by Software for LHCb- Resolved by Andre Gunther
@ahennequ I tested this yesterday with the
TrackListMerger
in Rec!3001 (closed) and observe a segfault, as soon as it tries to add the second container (and the segfault happens incopy_back
). The code looks very simple and it essentially copy-paste from some older code, so I was wondering if anything changed in the implementation of theSOACollection
.
- Resolved by Andre Gunther
/ci-test
added ci-test-triggered label
mentioned in merge request !3286 (merged)
added 209 commits
-
07aa39bb...cec9b7ca - 206 commits from branch
master
- 235ce558 - Add templated SOACollectionMerger, to make merging of SOACollection-based classes easier
- 8b8b2560 - fix some bugs
- e8be86b4 - add forgotten include
Toggle commit list-
07aa39bb...cec9b7ca - 206 commits from branch