Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • L LHCb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 197
    • Issues 197
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 46
    • Merge requests 46
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • LHCb
  • Merge requests
  • !3809

Add an is_barrier property to the PyConf algorithm wrapper

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Gerhard Raven requested to merge barrier-as-property into master Oct 10, 2022
  • Overview 13
  • Commits 1
  • Pipelines 17
  • Changes 2

... and use this property to discover which algorithm have to be declared to the scheduler as barriers, instead of having to pass their names explicitly through the call-stack.

From first-principles, whether or not an algorithm is (can be) used as a barrier is fixed by its implementation, specifically whether it can deal with optional inputs or not. Hence the natural way to do this is by adding an 'is_barrier' property to the PyConf algorithm wrapper, and use this to declare algorithms as barrier to the scheduler. (Note: ideally, setting of this property should happen fully automatically by recognizing whether the algorithm has optional inputs)

goes together with Allen!1026 (merged) and Moore!1814 (merged) and MooreAnalysis!93 (merged)

Edited Oct 13, 2022 by Gerhard Raven
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: barrier-as-property