Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 171
    • Merge requests 171
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !22066

Merged
Created Mar 20, 2019 by Yi Liu@yiliuContributor

ATLASRECTS-4582 MT migration of shared SplitClustersAmbiguityMap

  • Overview 64
  • Commits 8
  • Pipelines 8
  • Changes 31

link to Jira tickets:

  • https://its.cern.ch/jira/browse/ATLASRECTS-4582
  • https://its.cern.ch/jira/browse/ATLASRECTS-4308
  • https://its.cern.ch/jira/browse/ATLASRECTS-3038

recon meeting ppt
https://indico.cern.ch/event/795379/contributions/3304472/attachments/1790986/2917880/yiliu_190205.pdf

The 2 main issues:

  • SplitClustersAmbiguityMap WriteHandle and ReadHanle are both required by TrkAmbiguitySolver. It will confuse the scheduler.
  • SplitClustersAmbiguityMap is configured with same key name in 4 TrkAmbiguitySolver instances to share data in r21. However, in r22, it is not allowed to update SG object after the creation.

Potential solution:

  • split TrkAmbiguitySolver into 2 algorithms, one requires WriteHandle and the other requires ReadHandle.
  • Each TrkAmbiguitySolver instance (which was split into 2 algorithms) is configured SplitClustersAmbiguityMap key name.
  • Assuming the calling order of TrkAmbiguitySolver instances is fixed. Then the later instances access the earlier instance and merge the earlier SplitClustersAmbiguityMap into the new created SplitClustersAmbiguityMap.

The above idea has not been implemented in this WIP merge

Edited Aug 09, 2019 by Yi Liu
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: master-0319-split-solver