Skip to content
Snippets Groups Projects

Add templated mergers

Merged Michel De Cian requested to merge decianm-SOACollectionMerger into master

Add templated SOACollectionMerger and VectorOfObjectsMerger, to make merging of SOACollection-based classes and std::vector<Object> collections easier.

Edited by Michel De Cian

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
  • added RTA label

  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Michel De Cian resolved all threads

    resolved all threads

  • Michel De Cian mentioned in merge request Rec!3001 (closed)

    mentioned in merge request Rec!3001 (closed)

  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Edited by Software for LHCb
    • Author Developer
      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 in copy_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 the SOACollection.

  • Michel De Cian resolved all threads

    resolved all threads

  • Michel De Cian marked this merge request as ready

    marked this merge request as ready

  • Peilian Li mentioned in merge request !3286 (merged)

    mentioned in merge request !3286 (merged)

  • Michel De Cian added 209 commits

    added 209 commits

    Compare with previous version

  • Michel De Cian added 2 commits

    added 2 commits

    • 5a011a2e - ensure merged SOACollection has the same zip identifier as the first input
    • 811ef8bf - Add templated merger for std::vector<Object>

    Compare with previous version

  • Michel De Cian marked this merge request as draft

    marked this merge request as draft

  • Michel De Cian changed title from Add SOACollectionMerger to Draft: Add templated mergers

    changed title from Add SOACollectionMerger to Draft: Add templated mergers

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading