clean up

Author: chg-ext

bim2sim/plugins/PluginTEASER/bim2sim_teaser/examples/e1_simple_project_bps_teaser.py

@@ -16,8 +16,8 @@ def run_example_simple_building_teaser():
     """
     # Create a temp directory for the project, feel free to use a "normal"
     # directory
-    # ToDo set your project directory here
-    project_path = Path(r"/home/chris/src/bim2sim_up/projects/e1")
+    project_path = Path(tempfile.TemporaryDirectory(
+        prefix='bim2sim_teaser_example_e1_').name)
 
     # Set the ifc path to use and define which domain the IFC belongs to
     ifc_paths = {
@@ -50,8 +50,8 @@ def run_example_simple_building_teaser():
             Path(bim2sim.__file__).parent.parent /
             'test/resources/weather_files/DEU_NW_Aachen.105010_TMYx.mos')
     # Run a simulation directly with dymola after model creation
-    project.sim_settings.dymola_simulation = False
-    project.sim_settings.create_plots = False
+    project.sim_settings.dymola_simulation = True
+    project.sim_settings.create_plots = True
 
     # Select results to output:
     project.sim_settings.sim_results = [

bim2sim/project.py

@@ -49,7 +49,6 @@ def add_config_section(
                   if not callable(getattr(sim_settings, attr)) and not
                   attr.startswith('__')]
     for attr in attributes:
-        # ToDo (chg-ext): value(defaul) here maybe already overwritten?
         default_value = getattr(sim_settings, attr).value
         if isinstance(default_value, Enum):
             default_value = str(default_value)
@@ -422,7 +421,6 @@ def run(self, interactive=False, cleanup=True):
             raise AssertionError("Project ist not set correctly!")
 
         self.sim_settings.check_mandatory()
-
         success = False
         if interactive:
             run = self._run_interactive
@@ -506,7 +504,6 @@ def finalize(self, success=False):
             self.logger.info(f'Project "{self.name}" finished successful')
 
         # reset sim_settings:
-        # Todo (chg-ext): Why load default settings here
         self.playground.sim_settings.load_default_settings()
         # clean logger
         log.teardown_loggers()

bim2sim/tasks/base.py

@@ -75,7 +75,6 @@ class Playground:
     def __init__(self, project: Project):
         self.project = project
         self.sim_settings = project.plugin_cls.sim_settings()
-
         self.sim_settings.update_from_config(config=project.config)
         self.state = {}
         self.history = []

test/unit/tasks/__init__.py

@@ -51,7 +51,6 @@ def setUp(self) -> None:
     def tearDown(self) -> None:
         """Clean up after each test method."""
         self.helper.reset()
-        #self.playground.sim_settings.load_default_settings()
         self.playground.sim_settings.load_default_settings()
 
     def run_task(self, answers, reads):

test/unit/tasks/bps/test_enrich_use_cond.py

@@ -54,7 +54,6 @@ def test_enrichment_single_office_common_conditions(self):
 
     def test_enrichment_sim_setting_without_overwrite(self):
         """Tests enrichment of single office without overwrite via settings."""
-        # Todo (chg-ext): Side effect. Test runs separately, but not in context of the whole class
         print(self.playground.sim_settings.heating_tz_overwrite)
         print(self.playground.sim_settings.cooling_tz_overwrite)
         print(self.playground.sim_settings.ahu_tz_overwrite)