Merge pull request #86 from ubermag/dynamic_check

check system for dynamics and energy is added

micromagneticmodel/driver.py

@@ -50,6 +50,10 @@ def _schedule_commands(self, system, runner):
     def _read_data(self, system):
         """Update system with simulation output (magnetisation and scalar data)."""
 
+    @abc.abstractmethod
+    def _check_system(self, system):
+        """Check if the system contains all required information."""
+
     def drive(
         self,
         system,
@@ -123,7 +127,7 @@ def drive(
         # This method is implemented in the derived driver class. It raises
         # exception if any of the arguments are not valid.
         self.drive_kwargs_setup(kwargs)
-
+        self._check_system(system)
         workingdir = self._setup_working_directory(
             system=system, dirname=dirname, mode="drive", append=append
         )
@@ -233,7 +237,7 @@ def schedule(
         # This method is implemented in the derived driver class. It raises
         # exception if any of the arguments are not valid.
         self.schedule_kwargs_setup(kwargs)
-
+        self._check_system(system)
         workingdir = self._setup_working_directory(
             system=system, dirname=dirname, mode="drive", append=append
         )

micromagneticmodel/tests/test_driver.py

@@ -31,6 +31,9 @@ def schedule_kwargs_setup(self, schedule_kwargs):
     def drive_kwargs_setup(self, drive_kwargs):
         pass
 
+    def _check_system(self, system):
+        pass
+
     def _write_input_files(self, system, **kwargs):
         with open(f"{system.name}.input", "wt", encoding="utf-8") as f:
             f.write(str(-1))  # factor -1 used to invert magnetisation direction in call