GIT_REPOSITORIES/LPP/Users/mperrinel/SciQLop-fork Commit - r1242:162d6112469c

Updates states of synchronized variables when there is a move operation

Alexandre Leroux -

r1242:162d6112469c

parent child

Context file:

r1242:162d6112469c

plugins/amda/tests/FuzzingDefs.cpp +17 -1

                      return;
                  }
-                 // Unregisters variable from sync group: if there is no more variable in the group, resets the range
+                 // Unregisters variable from sync group: if there is no more variable in the group, resets the
+                 // range
                  auto &syncGroup = m_SyncGroupsPool.at(syncGroupId);
                  syncGroup.m_Variables.erase(variableId);
                  if (syncGroup.m_Variables.empty()) {
                  }
              }
+             void FuzzingState::updateRanges(VariableId variableId, const SqpRange &newRange)
+             {
+                 auto syncGroupId = this->syncGroupId(variableId);
+                 // Retrieves the variables to update:
+                 // - if the variable is synchronized to others, updates all synchronized variables
+                 // - otherwise, updates only the variable
+                 auto variablesToUpdate = syncGroupId.isNull() ? std::set<VariableId>{variableId}
+                                                               : m_SyncGroupsPool.at(syncGroupId).m_Variables;
+                 // Sets new range
+                 for (const auto &variableId : variablesToUpdate) {
+                     m_VariablesPool.at(variableId).m_Range = newRange;
+                 }
+             }

plugins/amda/tests/FuzzingDefs.h +4 0

                  /// @param syncGroupId the synchronization group from which to remove the variable
                  void desynchronizeVariable(VariableId variableId, SyncGroupId syncGroupId);
+                 /// Updates the range of a variable and all variables to which it is synchronized
+                 /// @param the variable for which to affect the range
+                 /// @param the range to affect
+                 void updateRanges(VariableId variableId, const SqpRange &newRange);
                  VariablesPool m_VariablesPool;
                  SyncGroupsPool m_SyncGroupsPool;

plugins/amda/tests/FuzzingOperations.cpp +4 -3

                      auto delta = RandomGenerator::instance().generateDouble(0, deltaMax);
                      // Moves variable to its new range
+                     auto isSynchronized = !fuzzingState.syncGroupId(variableId).isNull();
                      auto newVariableRange = SqpRange{m_RangeStartMoveFun(variableRange.m_TStart, delta),
                                                       m_RangeEndMoveFun(variableRange.m_TEnd, delta)};
                      qCInfo(LOG_FuzzingOperations()).noquote()
                          << "Performing" << m_Label << "on" << variable->name() << "(from" << variableRange
                          << "to" << newVariableRange << ")...";
-                     variableController.onRequestDataLoading({variable}, newVariableRange, false);
+                     variableController.onRequestDataLoading({variable}, newVariableRange, isSynchronized);
-                     // Updates variable's state
-                     variableState.m_Range = newVariableRange;
+                     // Updates state
+                     fuzzingState.updateRanges(variableId, newVariableRange);
                  }
                  MoveFunction m_RangeStartMoveFun;

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages