Merge remote-tracking branch 'origin/v2.1.0-dev' into always-dump-into-vc

CHANGELOG.md

@@ -43,6 +43,9 @@ will consitute of a breaking change warranting a new major release:
 - Disable missed deadlines per default. Not useful in orbit, and triggers all the time on the EM
   build after a number of subsequent runs, without any apparent reason (deadlines are not actually
   missed, thread usage displayed is nominal)
+- TM store dumpes will not be cancelled anymore if the transmitter is off. The dump can be cancelled
+  with an OFF command, and the PTME is perfectly capable of dumping without the transmitter being
+  on.
 - Transmitter state is not taken into account anymore for writing into the PTME. The PTME should
   be perfectly capable of generating a valid CADU, even when the transmitter is not ON for any
   reason.
@@ -71,6 +74,7 @@ will consitute of a breaking change warranting a new major release:
 - PLOC MPSoC special communication is now scheduled, which allows flash read and flash write
   commands to work.
 - Fixed the MPSoC flash write command.
+- Added missing ACS parameter.
 
 # [v2.0.5] 2023-05-11
 

mission/com/TmStoreTaskBase.cpp

@@ -94,13 +94,6 @@ void TmStoreTaskBase::cancelDump(DumpContext& ctx, PersistentTmStore& store, boo
 ReturnValue_t TmStoreTaskBase::handleOneDump(PersistentTmStoreWithTmQueue& store,
                                              DumpContext& dumpContext, bool& dumpPerformed) {
   ReturnValue_t result = returnvalue::OK;
-  // The PTME might have been reset an transmitter state change, so there is no point in continuing
-  // the dump.
-  // TODO: Will be solved in a cleaner way, this is kind of a hack.
-  if (not channel.isTxOn()) {
-    cancelDump(dumpContext, store, false);
-    return returnvalue::FAILED;
-  }
   // It is assumed that the PTME will only be locked for a short period (e.g. to change datarate).
   if (not channel.isBusy() and not ptmeLocked) {
     performDump(store, dumpContext, dumpPerformed);

mission/controller/acs/AcsParameters.cpp

@@ -539,6 +539,9 @@ ReturnValue_t AcsParameters::getParameter(uint8_t domainId, uint8_t parameterId,
         case 0xA:
           parameterWrapper->setVector(nadirModeControllerParameters.quatRef);
           break;
+        case 0xB:
+          parameterWrapper->setVector(nadirModeControllerParameters.refRotRate);
+          break;
         case 0xC:
           parameterWrapper->set(nadirModeControllerParameters.timeElapsedMax);
           break;