Fix test, actual number of points.

biosimulators · Aug 3, 2023 · e6bdc41 · e6bdc41
1 parent 356c953
commit e6bdc41
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/biosimulators_tellurium/core.py b/biosimulators_tellurium/core.py
@@ -270,7 +270,7 @@ def exec_sed_task(task, variables, preprocessed_task=None, log=None, config=None
             number_of_presim_points = round(number_of_presim_points) - sim.number_of_steps
             road_runner.simulate(sim.initial_time, sim.output_start_time, number_of_presim_points)
 
-        results = numpy.array(road_runner.simulate(sim.output_start_time, sim.output_end_time, sim.number_of_points).tolist()).transpose()
+        results = numpy.array(road_runner.simulate(sim.output_start_time, sim.output_end_time, sim.number_of_steps+1).tolist()).transpose()
     else:
         road_runner.steadyState()
         results = road_runner.getSteadyStateValues()

diff --git a/tests/test_core_main.py b/tests/test_core_main.py
@@ -436,12 +436,6 @@ def test_exec_sed_task_error_handling_with_biosimulators(self):
         with self.assertRaisesRegex(ValueError, 'targets are not supported'):
             variable_results, log = core.exec_sed_task(task, variables_2, simulator_config=simulator_config)
 
-        task_2 = copy.deepcopy(task)
-        task_2.simulation.output_start_time = 1.5
-        simulator_config.sedml_interpreter = SedmlInterpreter.biosimulators
-        with self.assertRaises(NotImplementedError):
-            variable_results, log = core.exec_sed_task(task_2, variables, simulator_config=simulator_config)
-
     def test_exec_sed_task_with_preprocesssed_task(self):
         # configure simulation
         task = sedml_data_model.Task(