Apply suggestions from code review

Author: pcarruscag

Common/include/CConfig.hpp

@@ -5943,9 +5943,10 @@ class CConfig {
    * \brief Set the translation rate of the marker.
    * \param[in] iDim - spatial component
    * \param[in] val - translational velocity
-   * \return Translational velocity of the mesh.
    */
-  void SetMarkerTranslation_Rate(unsigned short iMarkerMoving, unsigned short iDim, su2double val) const { MarkerTranslation_Rate[3*iMarkerMoving + iDim] = val;}
+  void SetMarkerTranslationRate(unsigned short iMarkerMoving, unsigned short iDim, su2double val) {
+    MarkerTranslation_Rate[3 * iMarkerMoving + iDim] = val;
+  }
 
   /*!
    * \brief Get the rotation rate of the mesh.
@@ -5969,14 +5970,16 @@ class CConfig {
    * \return Rotation velocity of the marker.
    */
   su2double GetMarkerRotationRate(unsigned short iMarkerMoving, unsigned short iDim) const { return MarkerRotation_Rate[3*iMarkerMoving + iDim];}
+
   /*!
    * \brief Set the rotation rate of the marker.
-   *  \param[in] iMarkerMoving -  Index of the moving marker (as specified in Marker_Moving)
+   * \param[in] iMarkerMoving -  Index of the moving marker (as specified in Marker_Moving)
    * \param[in] iDim - spatial component
    * \param[in] val - Rotational velocity
-
    */
-  void SetMarkerRotation_Rate(unsigned short iMarkerMoving, unsigned short iDim, su2double val) const { MarkerRotation_Rate[3*iMarkerMoving + iDim] = val;}
+  void SetMarkerRotation_Rate(unsigned short iMarkerMoving, unsigned short iDim, su2double val) {
+    MarkerRotation_Rate[3 * iMarkerMoving + iDim] = val;
+  }
 
   /*!
    * \brief Get the pitching rate of the mesh.

SU2_CFD/include/drivers/CDriver.hpp

@@ -542,15 +542,16 @@ class CDriver : public CDriverBase {
    * \param[in] rot_y - Value of Angular velocity about y-axes.
    * \param[in] rot_z - Value of Angular velocity about z-axes.
    */
-  void SetMarkerRotationRate(short int iMarker, passivedouble rot_x, passivedouble rot_y, passivedouble rot_z);
-    /*!
-   * \brief Set the moving wall marker rotation rates.
+  void SetMarkerRotationRate(unsigned short iMarker, passivedouble rot_x, passivedouble rot_y, passivedouble rot_z);
+
+  /*!
+   * \brief Set the moving wall marker translation rates.
    * \param[in] iMaker - Index of moving wall marker.
-   * \param[in] vel_x - Value of velocity about x-axes.
-   * \param[in] vel_y - Value of velocity about y-axes.
-   * \param[in] vel_z - Value of velocity about z-axes.
+   * \param[in] vel_x - Value of velocity along x-axis.
+   * \param[in] vel_y - Value of velocity along y-axis.
+   * \param[in] vel_z - Value of velocity along z-axis.
    */
-  void SetMarkerTranslationRate(short int iMarker, passivedouble vel_x, passivedouble vel_y, passivedouble vel_z);
+  void SetMarkerTranslationRate(unsigned short iMarker, passivedouble vel_x, passivedouble vel_y, passivedouble vel_z);
 
 /// \}
 };

SU2_CFD/src/iteration/CIteration.cpp

@@ -88,8 +88,6 @@ void CIteration::SetGrid_Movement(CGeometry** geometry, CSurfaceMovement* surfac
       }
 
       break;
-
-   
   }
 
   if (config->GetSurface_Movement(AEROELASTIC) || config->GetSurface_Movement(AEROELASTIC_RIGID_MOTION) || config->GetSurface_Movement(MOVING_WALL)) {
@@ -114,9 +112,10 @@ void CIteration::SetGrid_Movement(CGeometry** geometry, CSurfaceMovement* surfac
 
         grid_movement->UpdateMultiGrid(geometry, config);
       }
-      if(config->GetSurface_Movement(MOVING_WALL)){
-        geometry[MESH_0]->SetWallVelocity(config, true);
-    
+      if (config->GetSurface_Movement(MOVING_WALL)) {
+        for (auto iMGlevel = 0u; iMGlevel <= config->GetnMGLevels(); iMGlevel++) {
+          geometry[iMGlevel]->SetWallVelocity(config, iMGlevel == 0u);
+        }
       }
 
     }

SU2_CFD/src/python_wrapper_structure.cpp

@@ -166,14 +166,16 @@ void CDriver::SetRotationRate(passivedouble rot_x, passivedouble rot_y, passived
     config_container[iZone]->SetRotation_Rate(2, rot_z);
   }
 }
-void CDriver::SetMarkerRotationRate(short int iMarker, passivedouble rot_x, passivedouble rot_y, passivedouble rot_z){
+
+void CDriver::SetMarkerRotationRate(unsigned short iMarker, passivedouble rot_x, passivedouble rot_y, passivedouble rot_z) {
   for (iZone = 0; iZone < nZone; iZone++) {
     config_container[iZone]->SetMarkerRotation_Rate(iMarker, 0, rot_x);
     config_container[iZone]->SetMarkerRotation_Rate(iMarker, 1, rot_y);
     config_container[iZone]->SetMarkerRotation_Rate(iMarker, 2, rot_z);
   }
 }
-void CDriver::SetMarkerTranslationRate(short int iMarker, passivedouble vel_x, passivedouble vel_y, passivedouble vel_z){
+
+void CDriver::SetMarkerTranslationRate(unsigned short iMarker, passivedouble vel_x, passivedouble vel_y, passivedouble vel_z) {
   for (iZone = 0; iZone < nZone; iZone++) {
     config_container[iZone]->SetMarkerTranslation_Rate(iMarker, 0, vel_x);
     config_container[iZone]->SetMarkerTranslation_Rate(iMarker, 1, vel_y);