Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-35880

Send zeroed applied*Forces events when forces are zeroed

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_main_telescope
    • Labels:
      None

      Description

      M1M3 SS CSC doesn't send applied*Forces events when forces are zeroed - the zeroing is in fact handled in ForceController via calling disable method:

      void ForceController::zeroStaticForces() {
          SPDLOG_INFO("ForceController: zeroStaticForces()");
          if (_staticForceComponent.isEnabled()) {
              _staticForceComponent.disable();
          }
      }

      where disable method doesn't put zeroes into force vector, and doesn't call logApplied*Forces to publish the change. Fix that by moving zero*Forces into ForceComponent, add code to actually put 0 into message, and sends the message out so it gets logged into EFD.

      Also apply method check for enabled (and enable call) can be moved into force component, as when applying the forces, we would like to see them enabled:

      void ForceController::applyStaticForces() {
          SPDLOG_INFO("ForceController: applyStaticForces()");
          if (!_staticForceComponent.isEnabled()) {
              _staticForceComponent.enable();
          }
          _staticForceComponent.applyStaticForces(&_forceActuatorSettings->StaticXTable,
                                                  &_forceActuatorSettings->StaticYTable,
                                                  &_forceActuatorSettings->StaticZTable);
      }       
      

        Attachments

          Issue Links

            Activity

            No builds found.
            pkubanek Petr Kubanek created issue -
            pkubanek Petr Kubanek made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-35056 [ DM-35056 ]
            pkubanek Petr Kubanek made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            pkubanek Petr Kubanek made changes -
            Sprint TSSW Sprint - Aug 01 - Aug 15 [ 1182 ]
            Story Points 1
            pkubanek Petr Kubanek made changes -
            Epic Link DM-27720 [ 442125 ]
            Hide
            pkubanek Petr Kubanek added a comment -

            Please see GH PR.

            Show
            pkubanek Petr Kubanek added a comment - Please see GH PR.
            pkubanek Petr Kubanek made changes -
            Status In Progress [ 3 ] In Review [ 10004 ]
            wvreeven Wouter van Reeven made changes -
            Sprint TSSW Sprint - Aug 01 - Aug 15 [ 1182 ] TSSW Sprint - Aug 01 - Aug 15, TSSW Sprint - Aug 15 - Aug 29 [ 1182, 1183 ]
            Hide
            pkubanek Petr Kubanek added a comment -

            Reviewd in github

            Show
            pkubanek Petr Kubanek added a comment - Reviewd in github
            pkubanek Petr Kubanek made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            pkubanek Petr Kubanek made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            aclements Andy Clements made changes -
            Epic Link DM-27720 [ 442125 ] DM-27721 [ 442128 ]

              People

              Assignee:
              pkubanek Petr Kubanek
              Reporter:
              pkubanek Petr Kubanek
              Reviewers:
              Dave Mills
              Watchers:
              Dave Mills, Doug Neill, Felipe Daruich, Petr Kubanek
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.