Revert "Modal idf (#360)"

This reverts commit d212716.

funtofem/driver/__init__.py

@@ -12,15 +12,6 @@
 from .funtofem_nlbgs_driver import *
 from .funtofem_nlbgs_fsi_subiters_driver import *
 from .transfer_settings import *
-import importlib
-caps_loader = importlib.util.find_spec("pyCAPS")
-if caps_loader is not None:
-    from .funtofem_shape_driver import *
+from .funtofem_shape_driver import *
 from .oneway_struct_driver import *
 from .oneway_aero_driver import *
-
-# modal IDF driver
-from .modal_idf_driver import *
-
-# import all the custom or special drivers
-from .custom import *

funtofem/driver/_funtofem_driver.py

@@ -70,14 +70,6 @@ def __init__(
             whether to save and reload funtofem states
         """
 
-        # assert at least one coupled scenario
-        self.coupled_scenarios = []
-        if model is not None: # only case where model is None is fakeModel?
-            any_coupled = any([scenario.coupled for scenario in model.scenarios])
-            assert any_coupled
-
-            self.coupled_scenarios = [scenario for scenario in model.scenarios if scenario.coupled]
-
         # add the comm manger
         if comm_manager is not None:
             comm_manager = comm_manager
@@ -175,8 +167,7 @@ def solve_forward(self, steps=None):
             body.update_shape(complex_run)
 
         # loop over the forward problem for the different scenarios
-        for scenario in self.coupled_scenarios:
-
+        for scenario in self.model.scenarios:
             # tell the solvers what the variable values and functions are for this scenario
             if not self.fakemodel:
                 self._distribute_variables(scenario, self.model.bodies)
@@ -232,10 +223,11 @@ def solve_adjoint(self):
 
         # Zero the derivative values stored in the function
         self._zero_derivatives()
+        for func in functions:
+            func.zero_derivatives()
 
         # Set the functions into the solvers
-        for scenario in self.coupled_scenarios:
-
+        for scenario in self.model.scenarios:
             # tell the solvers what the variable values and functions are for this scenario
             self._distribute_variables(scenario, self.model.bodies)
             self._distribute_functions(scenario, self.model.bodies)
@@ -296,11 +288,9 @@ def _initialize_adjoint(self, scenario, bodies):
 
     def _zero_derivatives(self):
         """zero all model derivatives"""
-        # TODO : only zero derivatives in coupled scenarios when using
-        for scenario in self.coupled_scenarios: # no need to zero composite functions, they are exactly diff later with no += effects
-            for func in scenario.functions:
-                for var in self.model.get_variables():
-                    func.derivatives[var] = 0.0
+        for func in self.model.get_functions(all=True):
+            for var in self.model.get_variables():
+                func.derivatives[var] = 0.0
         return
 
     def _post_forward(self, scenario, bodies):

funtofem/driver/funtofem_shape_driver.py

@@ -215,7 +215,7 @@ def __init__(
                     self._flow_solver_type = "fun3d"
             # TBD on new types
         else:  # check with shape change
-            if fun3d_loader is not None and caps_loader is not None:
+            if fun3d_loader is not None:
                 if isinstance(model.flow, Fun3dModel):
                     self._flow_solver_type = "fun3d"
                     self.flow_aim = model.flow.fun3d_aim