Uploaded image for project: 'SAFe Program'
  1. SAFe Program
  2. SP-2077

Refactor and release long running command implementation in ska-tango-base

Change Owns to Parent OfsSet start and due date...
    XporterXMLWordPrintable

Details

    • Obs Mgt & Controls
    • Hide

      Refactoring relevant areas of the ska-tango-base codebase according to the MCCS-852 proposal better accommodates devices which are themselves asynchronous in nature and better separates interfaces from concurrency mechanisms.

      Show
      Refactoring relevant areas of the ska-tango-base codebase according to the MCCS-852 proposal better accommodates devices which are themselves asynchronous in nature and better separates interfaces from concurrency mechanisms.
    • Hide
      • LRC Interface updated according to MCCS-852: Proposal for SP-2077, Long Running Commands
        • Modify the implementation to better support LRC payloads that are themselves implemented asynchronously.
        • Decouple the Tango interface for long-running commands from the concurrency mechanism used to deliver them.
           
      • Tests updated and passing
      • Documentation updated with relevant examples
      • Base and Subarray devices
      • New version released and available from CAR
      Show
      LRC Interface updated according to  MCCS-852: Proposal for SP-2077, Long Running Commands Modify the implementation to better support LRC payloads that are themselves implemented asynchronously. Decouple the Tango interface for long-running commands from the concurrency mechanism used to deliver them.   Tests updated and passing Documentation updated with relevant examples Base and Subarray devices New version released and available from CAR
    • 3
    • 3
    • 40
    • 13.333
    • Team_KAROO, Team_MCCS
    • Sprint 5
    • Hide

      Coding for this feature was conducted on the MCCS-876 branch. In the merge request for this branch you will find an explanation and rationale for the design.

      The Tango layer has been decoupled from the concurrency mechanism (MCCS-878). This gives flexibility in the implementation of concurrency.

      A command tracker (MCCS-876) allows the Tango layer to track the status of command execution,, without being coupled to the implementation.

      Component managers now use call-backs to indicate change of state. This allows the decoupling of the LRC interface from the implementation (MCCS-877). Such an explicit signalling mechanism removes the reliance on the implicit signalling of thread completion, better to permit the use of payloads which are themselves implemented asynchronously (MCCS-879).

      After review of this work (SAR-336) tests and type hinting were completed (SAR-351) and the documentation brought up to date (SAR-341).

      The updated Tango base classes were released to CAR as version 0.13.0 (artefact links here)

      Show
      Coding for this feature was conducted on the MCCS-876 branch. In the merge request for this branch you will find an explanation and rationale for the design. The Tango layer has been decoupled from the concurrency mechanism (MCCS-878). This gives flexibility in the implementation of concurrency. A command tracker (MCCS-876) allows the Tango layer to track the status of command execution,, without being coupled to the implementation. Component managers now use call-backs to indicate change of state. This allows the decoupling of the LRC interface from the implementation (MCCS-877). Such an explicit signalling mechanism removes the reliance on the implicit signalling of thread completion, better to permit the use of payloads which are themselves implemented asynchronously (MCCS-879). After review of this work (SAR-336) tests and type hinting were completed (SAR-351) and the documentation brought up to date (SAR-341). The updated Tango base classes were released to CAR as version 0.13.0 (artefact links here )
    • 13.6
    • Stories Completed, Solution Intent Updated, Outcomes Reviewed, Demonstrated, Satisfies Acceptance Criteria, Accepted by FO

    • PI24 - UNCOVERED

    • Cross_Team_Code_Review SS-54 Team_KAROO mccs_software

    Description

      During PI11 the KAROO team investigated how ska-tango-base could be updated to implement a solution to address the long running commands (LRC) issue that has been identified in various areas of the SKA software community.  The solution identified was implemented in PI12, and the MCCS team adopted the approach to prove it and identify learnings and recommend any further changes. This feature is to refactor the LRC implementation in light of those learnings / recommendations, and finally to release and socialise the solution.

      The specific refactoring work identified are:

      • Modify the implementation to better support LRC payloads that are themselves implemented asynchronously.
      • Decouple the Tango interface for long-running commands from the concurrency mechanism used to deliver them.

      For details see https://confluence.skatelescope.org/display/SE/MCCS-852%3A+Proposal+for+SP-2077.

      Attachments

        Issue Links

          depends on

          Enabler - An Enabler supports the activities needed to extend the Architectural Runway to provide future business functionality. These include exploration, architecture, infrastructure, and compliance. SP-1627 TANGO Base Class Controller implements long-running commands that trigger operational state transitions

          • Must have - Items labelled as "Must have" are critical to the current delivery timebox in order for it to be a success.
          • Done

          Enabler - An Enabler supports the activities needed to extend the Architectural Runway to provide future business functionality. These include exploration, architecture, infrastructure, and compliance. SP-1628 TANGO Base Class Subarray implements long running commands that trigger observing state transitions

          • Must have - Items labelled as "Must have" are critical to the current delivery timebox in order for it to be a success.
          • Done

          Enabler - An Enabler supports the activities needed to extend the Architectural Runway to provide future business functionality. These include exploration, architecture, infrastructure, and compliance. SP-1827 Update TANGO Base Classes to handle long running commands

          • Must have - Items labelled as "Must have" are critical to the current delivery timebox in order for it to be a success.
          • Done
          informs

          Architectural Decision - Software Architectural Decision ADR-47 Guidelines for implementation of commands

          • decided
          is required by

          Feature - A Feature is a service that fulfills a stakeholder need. Each feature includes a benefit hypothesis and acceptance criteria, and is sized or split as necessary to be delivered by a single Agile Release Train (ART) in a Program Increment (PI). SP-2586 MCCS adoption of ska_base_classes v0.13. post adoption clean up and demo

          • Must have - Items labelled as "Must have" are critical to the current delivery timebox in order for it to be a success.
          • Done

          Feature - A Feature is a service that fulfills a stakeholder need. Each feature includes a benefit hypothesis and acceptance criteria, and is sized or split as necessary to be delivered by a single Agile Release Train (ART) in a Program Increment (PI). SP-2813 Make a post-base-classes-0.13 release of MCCS

          • Must have - Items labelled as "Must have" are critical to the current delivery timebox in order for it to be a success.
          • Done

          Feature - A Feature is a service that fulfills a stakeholder need. Each feature includes a benefit hypothesis and acceptance criteria, and is sized or split as necessary to be delivered by a single Agile Release Train (ART) in a Program Increment (PI). SP-2251 MCCS adoption of ska_base_classes v0.13.

          • Should have - Items labelled as "Should have" are important but not necessary for delivery in the current delivery timebox. While "Should have" items can be as important as "Must have", they are often not as time-critical or there may be another way to satisfy the requirement so that it can be held back until a future delivery timebox.
          • Done

          Enabler - An Enabler supports the activities needed to extend the Architectural Runway to provide future business functionality. These include exploration, architecture, infrastructure, and compliance. SS-54 Review of Control Systems guidelines and refactoring of implementation

          • Not Assigned - Priority is not assigned yet.
          • Done
          relates to

          Enabler - An Enabler supports the activities needed to extend the Architectural Runway to provide future business functionality. These include exploration, architecture, infrastructure, and compliance. SS-54 Review of Control Systems guidelines and refactoring of implementation

          • Not Assigned - Priority is not assigned yet.
          • Done
          mentioned in

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          mentioned on

          GitLab Commit - Merge branch 'mccs-876-lrc-refactor' into 'main'

          GitLab Commit - SP-1984 Use pybind11_mkdoc to extract docstrings

          GitLab Merge request - [MCCS-876] LRC refactor

          Structure

            Activity

              People

                s.vrcic Vrcic, Sonja
                r.braddock Braddock, Ralph
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Feature Progress

                  Story Point Burn-up: (100.00%)

                  Feature Estimate: 3.0

                  IssuesStory Points
                  To Do00.0
                  In Progress   00.0
                  Complete920.5
                  Total920.5

                  Dates

                    Created:
                    Updated:
                    Resolved:

                    Structure Helper Panel