Bump dependency minimum versions

.pre-commit-config.yaml

@@ -44,7 +44,7 @@ repos:
         pass_filenames: false
   # https://docs.astral.sh/ruff/integrations/#pre-commit
   - repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.8.5
+    rev: v0.9.1
     hooks:
       # Run the linter
       - id: ruff

geojson_modelica_translator/model_connectors/load_connectors/teaser.py

@@ -288,8 +288,7 @@ def post_process(self, scaffold, keep_original_models=False):
                 "Boolean",
                 "use_moisture_balance",
                 assigned_value=use_moisture_balance,
-                string_comment="If true, input connector QLat_flow is enabled and room air computes"
-                " moisture balance.",
+                string_comment="If true, input connector QLat_flow is enabled and room air computes moisture balance.",
             )
             # create a integer parameter to evaluate number of connected ports.
             mofile.add_parameter(
@@ -476,7 +475,7 @@ def post_process(self, scaffold, keep_original_models=False):
 
             # Save as the new filename (without building ID)
             new_filename = os.path.join(
-                scaffold.loads_path.files_dir, f'{self.building_name}/{os.path.basename(f).split("_")[1]}'
+                scaffold.loads_path.files_dir, f"{self.building_name}/{os.path.basename(f).split('_')[1]}"
             )
             mofile.save_as(new_filename)
             os.remove(f)

geojson_modelica_translator/model_connectors/plants/borefield.py

@@ -116,9 +116,9 @@ def to_modelica(self, scaffold):
             except FileNotFoundError:
                 raise SystemExit(
                     f"Can't find g-function file for ghe with ID: {template_data['gfunction']['ghe_id']}.\n"
-                    'If using a relative path to your ghe_dir, your path '
-                    f' \'{template_data["gfunction"]["input_path"]}\' must be relative to the dir your '
-                    'sys-param file is in.'
+                    "If using a relative path to your ghe_dir, your path "
+                    f" '{template_data['gfunction']['input_path']}' must be relative to the dir your "
+                    "sys-param file is in."
                 )
         template_data["gfunction"]["gfunction_file_rows"] = gfunction.shape[0] + 1
 

geojson_modelica_translator/model_connectors/plants/templates/GroundTemperatureResponse.mo

@@ -18,7 +18,7 @@ model GroundTemperatureResponse
   parameter Integer nTimTot=76
     "Total length of g-function vector";
   Boolean writegFun
-    "True if g-function was succesfully written to file";
+    "True if g-function was successfully written to file";
   Modelica.Blocks.Interfaces.RealOutput delTBor(
     unit="K")
     "Temperature difference current borehole wall temperature minus initial borehole wall temperature"

geojson_modelica_translator/modelica/csv_modelica.py

@@ -146,7 +146,7 @@ def timeseries_to_modelica_data(
             line2 = f"{data_type} {output_modelica_file_name}({size[0]}, {size[1]})"
             line3 = "#Nominal heating water mass flow rate=" + str(self.nominal_heating_mass_flow_rate.loc[0, "#value"])
             line4 = "#Nominal chilled water mass flow rate=" + str(self.nominal_cooling_mass_flow_rate.loc[0, "#value"])
-            f.write(f"{line1}\n" f"{line2}\n" f"{line3}\n" f"{line4}\n")
+            f.write(f"{line1}\n{line2}\n{line3}\n{line4}\n")
             if "SecondsFromStart" in self.timeseries_output.columns:
                 self.timeseries_output.to_csv(f, header=True, index=False)
             else:

geojson_modelica_translator/system_parameters/system_parameters.py

@@ -298,7 +298,7 @@ def download_weatherfile(self, filename, save_directory: str) -> Union[str, Path
                     f"Returned non 200 status code trying to download weather file: {weatherfile_data.status_code}"
                 )
         except requests.exceptions.RequestException as e:
-            raise requests.exceptions.RequestException(f"Could not download weather file: {weatherfile_url}" f"\n{e}")
+            raise requests.exceptions.RequestException(f"Could not download weather file: {weatherfile_url}\n{e}")
 
         if not outputname.exists():
             raise FileNotFoundError(f"Could not find or download weather file for {p_download!s}")

management/check_sys_params.py

@@ -240,11 +240,11 @@ def check_sys_params(v, vv, output_file, log_file):
         for idx, param in enumerate(params):
             print(delimiter, file=output_file_handle)
             if param["skip"]:
-                print(f'{param["path"]} Skipped (duplicate)', file=output_file_handle)
+                print(f"{param['path']} Skipped (duplicate)", file=output_file_handle)
                 continue
 
             if param["type"] == "string":
-                print(f'{param["path"]} Skipped (string)', file=output_file_handle)
+                print(f"{param['path']} Skipped (string)", file=output_file_handle)
                 continue
 
             jsonpath = parse(param["path"])
@@ -269,7 +269,7 @@ def check_sys_params(v, vv, output_file, log_file):
                 test_sys_params_path = fp.name
                 create_package(test_package_path, baseline_geojson_path, test_sys_params_path)
 
-            print(f'{param["path"]}   Testing    Original: {original_value}; New: {value}', file=output_file_handle)
+            print(f"{param['path']}   Testing    Original: {original_value}; New: {value}", file=output_file_handle)
 
             # compare the package to baseline (find diffs)
             print_diffs(baseline_package_path, test_package_path, output_file_handle)

management/format_modelica_files.py

@@ -4,33 +4,19 @@
 import os
 import re
 import subprocess
+import sys
 from pathlib import Path
 from tempfile import mkstemp
 
 import click
 
-SKIP_FILES = ["DistrictEnergySystem.mot", "DistrictEnergySystem5G.mot"]
-TEMPLATE_FILES = Path("geojson_modelica_translator/model_connectors").glob("**/templates/*")
+project_root = os.path.dirname(os.path.abspath(__file__))
+if project_root not in sys.path:
+    sys.path.append(project_root)
 
 
-@click.command()
-@click.argument("mofile", required=False)
-def fmt_modelica_files(mofile):
-    if mofile is not None:
-        files = [mofile]
-    else:
-        files = TEMPLATE_FILES
-
-    for filepath in files:
-        if os.path.basename(filepath) in SKIP_FILES:
-            continue
-        try:
-            if filepath.suffix == ".mot":
-                preprocess_and_format(str(filepath))
-            elif filepath.suffix == ".mo":
-                apply_formatter(str(filepath))
-        except FormattingError as e:
-            click.echo(f"Error processing file {filepath}:\n    {e}", err=True)
+SKIP_FILES = ["DistrictEnergySystem.mot", "DistrictEnergySystem5G.mot"]
+TEMPLATE_FILES = Path("geojson_modelica_translator/model_connectors").glob("**/templates/*")
 
 
 class FormattingError(Exception):
@@ -97,7 +83,7 @@ def sub_generic(text, sub_map):
     matches = reversed([m.span() for m in GENERIC_CONTROL_REGEX.finditer(text)])
     for span in matches:
         sub_id = sub_map.add_sub(text[span[0] : span[1]])
-        text = f"{text[:span[0]]}/*{sub_id}*/{text[span[1]:]}"
+        text = f"{text[: span[0]]}/*{sub_id}*/{text[span[1] :]}"
 
     return text
 
@@ -115,7 +101,7 @@ def sub_expression(text, sub_map):
     matches = reversed([m.span() for m in EXPRESSION_REGEX.finditer(text)])
     for span in matches:
         sub_id = sub_map.add_sub(text[span[0] : span[1]])
-        text = f"{text[:span[0]]}{sub_id}{text[span[1]:]}"
+        text = f"{text[: span[0]]}{sub_id}{text[span[1] :]}"
 
     return text
 
@@ -201,3 +187,35 @@ def preprocess_and_format(filename, outfilename=None):
             f.write(formatted_result)
     finally:
         os.remove(tmp_filepath)
+
+
+@click.command()
+@click.argument("mofile", required=False)
+@click.argument("debug", required=False)
+def format_modelica_files(mofile=None, debug=False):
+    if mofile is not None:
+        files = [mofile]
+    else:
+        files = TEMPLATE_FILES
+
+    for filepath in files:
+        if os.path.basename(filepath) in SKIP_FILES:
+            continue
+        try:
+            print(f"Formatting mo/mot file: {filepath}")
+            # Can't format mopt yet.
+            if filepath.suffix == ".mot":
+                if debug:
+                    print(f"Formatting mot file: {filepath}")
+                preprocess_and_format(str(filepath))
+            elif filepath.suffix == ".mo":
+                if debug:
+                    print(f"Formatting mo file: {filepath}")
+                apply_formatter(str(filepath))
+        except FormattingError as e:
+            click.echo(f"Error processing file {filepath}:\n    {e}", err=True)
+
+
+if __name__ == "__main__":
+    """Method to call this python script manually to format files"""
+    format_modelica_files(mofile=None, debug=True)