Skip to content
Snippets Groups Projects

Add HitHandler for PrUTHits, templated PrStoreUTHit, added PrModUTHit and adapted PrVeloUT

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

Add HitHandler for SoA hits (PrUTHits), template PrStoreUTHit to use the old and new hits concurrently for the time being. Introduced an SoA modifiable UT hit (PrModUTHits), adapted PrVeloUT to use some vectorization. Needs LHCb!2204 (merged).

Needed for Moore!314 (merged)

Edited by Sascha Stahl

Merge request reports

Pipeline #1308498 passed

Pipeline passed for d7271e2e on decianm-PrUTSOAHits

Approved by

Merged by Rosen MatevRosen Matev 5 years ago (Jan 13, 2020 1:51pm UTC)

Merge details

  • Changes merged into master with 3c55e85c.
  • Deleted the source branch.

Pipeline #1339276 passed

Pipeline passed for 3c55e85c on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
269 270 mutHits.compressstore_weight( index, mask, myHits.weight<simd::float_v>( i ) );
270 271 mutHits.compressstore_channelID( index, mask, myHits.channelID<simd::int_v>( i ) );
271 272 mutHits.size += simd::popcount( mask );
272 ;
273
274 if( mutHits.size > LHCb::Pr::UT::Mut::Hits::max_hits - simd::size ) break;
  • Let's add this to lhcb-head, along with !2204 (merged), and see how it does.

    @decianm please un-WIP if this is ready to go (assuming !2204 (merged) is good).

  • added lhcb-head label

  • Michel De Cian added 19 commits

    added 19 commits

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    Compare with previous version

  • Michel De Cian added 1 commit

    added 1 commit

    • 3a1e1c80 - add error message if max hits are reached, fix artifacts from previous manual merge

    Compare with previous version

  • Edited by Software for LHCb
  • mentioned in merge request LHCb!2204 (merged)

  • Michel De Cian added 1 commit

    added 1 commit

    • 9acc893d - fix protection against too many hits

    Compare with previous version

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