Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • Gaudi Gaudi
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 104
    • Issues 104
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 29
    • Merge requests 29
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Gaudi
  • GaudiGaudi
  • Merge requests
  • !316

Algorithm clonability should be explicitly set by Alg author

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Charles Leggett requested to merge leggett/Gaudi:dev/Clonable into master Apr 04, 2017
  • Overview 32
  • Commits 9
  • Changes 25

The clonability of an Algorithm is best decided by the Algorithm author, and should not be a Property of the Algorithm that can be changed at run time.

This MR removes the IsClonable Property of Algorithms, and changes it into a protected bool that derived Algs can modify. We should consider whether the default should be true or false. It is currently set to true for the sake of convenience.

If an Alg is not clonable, then its cardinality is automatically set to 1 during sysInitialize.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: dev/Clonable