power-grid-model-io 1.3.5__tar.gz → 1.3.6__tar.gz

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: power-grid-model-io
-Version: 1.3.5
+Version: 1.3.6
 Summary: Power Grid Model Input/Output
 Author-email: Contributors to the Power Grid Model project <powergridmodel@lfenergy.org>
 License: MPL-2.0
@@ -33,6 +33,7 @@ Requires-Dist: pre-commit; extra == "dev"
 Requires-Dist: pylint; extra == "dev"
 Requires-Dist: pytest; extra == "dev"
 Requires-Dist: pytest-cov; extra == "dev"
+Requires-Dist: ruff; extra == "dev"
 Requires-Dist: pydantic>2; extra == "dev"
 Requires-Dist: pandapower>2.11.1; extra == "dev"
 Provides-Extra: examples

power_grid_model_io-1.3.6/PYPI_VERSION

@@ -0,0 +1 @@
+1.3.6

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/pyproject.toml

@@ -44,6 +44,7 @@ dev = [
     "pylint",
     "pytest",
     "pytest-cov",
+    "ruff",
     "pydantic>2", # Used in unit tests
     "pandapower>2.11.1",
 ]
@@ -89,33 +90,43 @@ addopts = [
 ]
 xfail_strict = true
 
-[tool.black]
+[tool.ruff]
+# Same as Black.
 line-length = 120
-target-version = ["py311", "py312", "py313"]
+indent-width = 4
+show-fixes = true
 
-[tool.isort]
-profile = "black"
-line_length = 120
+# Assume Python 3.11
+target-version = "py311"
 
-[tool.pylint]
-max-line-length = 120
-ignore-paths = [
-    "docs/",
-    "examples/",
-    "tests/",
-    "setup.py",
-]
-disable = [
-    "fixme", # allow todos
-]
-good-names=[
-    "ex",   # exception
-    "i",    # iterator or current
-    "p",    # active power
-    "q",    # reactive power
-    "s",    # power (p + q)
-    "v",    # voltage
+[tool.ruff.lint]
+select = [
+    # pycodestyle
+    "E",
+    # Pyflakes
+    "F",
+    # isort
+    "I",
+    "SIM",
+    "YTT",
+    "BLE",
+    "PERF",
+    "Q",
+    "ICN",
+    "ISC",
+    "G",
+    "LOG",
+    "EXE",
+    "FA",
+    "FURB",
+    "FLY",
+    "SLOT",
 ]
+ignore = ["SIM108", "SIM118", "SIM110", "SIM211"]
+
+[tool.ruff.lint.isort]
+# Imports that are imported using keyword "as" and are from the same source - are combined.
+combine-as-imports = true
 
 [tool.mypy]
 follow_imports = "silent"

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/converters/__init__.py

@@ -8,3 +8,5 @@ Converters
 from power_grid_model_io.converters.pandapower_converter import PandaPowerConverter
 from power_grid_model_io.converters.pgm_json_converter import PgmJsonConverter
 from power_grid_model_io.converters.vision_excel_converter import VisionExcelConverter
+
+__all__ = ["PandaPowerConverter", "PgmJsonConverter", "VisionExcelConverter"]

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/converters/base_converter.py

@@ -4,6 +4,7 @@
 """
 Abstract converter class
 """
+
 import logging
 from abc import ABC, abstractmethod
 from typing import Generic, Optional, Tuple, TypeVar

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/converters/pandapower_converter.py

@@ -461,7 +461,7 @@ class PandaPowerConverter(BaseConverter[PandaPowerData]):
         }
         if not all(checks.values()):
             failed_checks = ", ".join([key for key, value in checks.items() if not value])
-            logger.warning(f"Zero sequence parameters given in external grid shall be ignored:{failed_checks}")
+            logger.warning("Zero sequence parameters given in external grid shall be ignored: %s", failed_checks)
 
         pgm_sources = initialize_array(data_type="input", component_type="source", shape=len(pp_ext_grid))
         pgm_sources["id"] = self._generate_ids("ext_grid", pp_ext_grid.index)
@@ -753,7 +753,7 @@ class PandaPowerConverter(BaseConverter[PandaPowerData]):
         }
         if not all(checks.values()):
             failed_checks = ", ".join([key for key, value in checks.items() if not value])
-            logger.warning(f"Zero sequence parameters given in trafo shall be ignored:{failed_checks}")
+            logger.warning("Zero sequence parameters given in trafo shall be ignored: %s", failed_checks)
 
         # Do not use taps when mandatory tap data is not available
         no_taps = np.equal(tap_side, None) | np.isnan(tap_pos) | np.isnan(tap_nom) | np.isnan(tap_size)
@@ -862,7 +862,7 @@ class PandaPowerConverter(BaseConverter[PandaPowerData]):
         }
         if not all(checks.values()):
             failed_checks = ", ".join([key for key, value in checks.items() if not value])
-            logger.warning(f"Zero sequence parameters given in trafo3w are ignored: {failed_checks}")
+            logger.warning("Zero sequence parameters given in trafo3w are ignored: %s", failed_checks)
 
         # Do not use taps when mandatory tap data is not available
         no_taps = np.equal(tap_side, None) | np.isnan(tap_pos) | np.isnan(tap_nom) | np.isnan(tap_size)

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/converters/pgm_json_converter.py

@@ -238,7 +238,7 @@ class PgmJsonConverter(BaseConverter[StructuredData]):
         """
 
         # This should be a single data set
-        for component, array in data.items():
+        for array in data.values():
             if not isinstance(array, np.ndarray) or array.ndim != 1:
                 raise ValueError("Invalid data format")
 

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/data_stores/vision_excel_file_store.py

@@ -4,6 +4,7 @@
 """
 Vision Excel file store
 """
+
 from pathlib import Path
 from typing import Optional
 

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/data_types/__init__.py

@@ -7,3 +7,5 @@ Common data types used in the Power Grid Model project
 
 from power_grid_model_io.data_types._data_types import ExtraInfo, ExtraInfoLookup, StructuredData
 from power_grid_model_io.data_types.tabular_data import LazyDataFrame, TabularData
+
+__all__ = ["ExtraInfo", "ExtraInfoLookup", "StructuredData", "LazyDataFrame", "TabularData"]

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/functions/__init__.py

@@ -17,3 +17,15 @@ from power_grid_model_io.functions._functions import (
     value_or_default,
     value_or_zero,
 )
+
+__all__ = [
+    "both_zeros_to_nan",
+    "complex_inverse_imaginary_part",
+    "complex_inverse_real_part",
+    "degrees_to_clock",
+    "get_winding",
+    "has_value",
+    "is_greater_than",
+    "value_or_default",
+    "value_or_zero",
+]

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/mappings/tabular_mapping.py

@@ -4,6 +4,7 @@
 """
 Tabular data mapping helper class
 """
+
 from typing import Dict, Generator, List, Tuple, Union
 
 import structlog

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/mappings/value_mapping.py

@@ -4,6 +4,7 @@
 """
 Value substitution helper class
 """
+
 from typing import Dict, Optional, Union
 
 import structlog

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/utils/auto_id.py

@@ -4,6 +4,7 @@
 """
 Automatic ID generator class
 """
+
 import collections
 from collections.abc import Hashable
 from typing import Any, Dict, List, Optional, Union

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/utils/dict.py

@@ -4,6 +4,7 @@
 """
 General dictionary utilities
 """
+
 from copy import deepcopy
 from typing import Dict
 

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/utils/excel_ambiguity_checker.py

@@ -19,6 +19,7 @@ Requirements:
     - xml.etree.ElementTree for parsing XML structures within the Excel file.
     - zipfile to handle the Excel file as a ZIP archive for parsing.
 """
+
 import os
 import xml.etree.ElementTree as ET
 import zipfile
@@ -80,11 +81,10 @@ class ExcelAmbiguityChecker:
             list: A list of shared strings used in the Excel file.
         """
         shared_strings_path = SHARED_STR_PATH
-        shared_strings = []
+        shared_strings: List[Optional[str]] = []
         with zip_file.open(shared_strings_path) as f:
             tree = ET.parse(f)
-            for si in tree.findall(FIND_T, namespaces=XML_NAME_SPACE):
-                shared_strings.append(si.text)
+            shared_strings.extend(si.text for si in tree.findall(FIND_T, namespaces=XML_NAME_SPACE))
         return shared_strings
 
     def _get_column_names_from_row(self, row, shared_strings) -> List[Optional[str]]:

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io/utils/modules.py

@@ -4,6 +4,7 @@
 """
 Module utilities, expecially useful for loading optional dependencies
 """
+
 from importlib import import_module
 from typing import Callable
 

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: power-grid-model-io
-Version: 1.3.5
+Version: 1.3.6
 Summary: Power Grid Model Input/Output
 Author-email: Contributors to the Power Grid Model project <powergridmodel@lfenergy.org>
 License: MPL-2.0
@@ -33,6 +33,7 @@ Requires-Dist: pre-commit; extra == "dev"
 Requires-Dist: pylint; extra == "dev"
 Requires-Dist: pytest; extra == "dev"
 Requires-Dist: pytest-cov; extra == "dev"
+Requires-Dist: ruff; extra == "dev"
 Requires-Dist: pydantic>2; extra == "dev"
 Requires-Dist: pandapower>2.11.1; extra == "dev"
 Provides-Extra: examples

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/src/power_grid_model_io.egg-info/requires.txt

@@ -11,6 +11,7 @@ pre-commit
 pylint
 pytest
 pytest-cov
+ruff
 pydantic>2
 pandapower>2.11.1
 

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/conftest.py

@@ -2,9 +2,9 @@
 #
 # SPDX-License-Identifier: MPL-2.0
 
+from contextlib import suppress
+
 import pandas as pd
 
-try:
+with suppress(pd.errors.OptionError):
     pd.set_option("future.no_silent_downcasting", True)
-except pd.errors.OptionError:
-    pass

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/unit/conftest.py

@@ -2,10 +2,10 @@
 #
 # SPDX-License-Identifier: MPL-2.0
 
+from contextlib import suppress
+
 import pandas as pd
 
-try:
+with suppress(pd.errors.OptionError):
     # TODO(mgovers) We're ready for Pandas 3.x, but pandapower is not. Move to parent conftest when it is.
     pd.set_option("mode.copy_on_write", True)
-except pd.errors.OptionError:
-    pass

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/unit/converters/test_tabular_converter.py

@@ -233,7 +233,7 @@ def test_convert_col_def_to_attribute(
 ):
     with pytest.raises(
         KeyError,
-        match=r"Could not find attribute 'incorrect_attribute' for 'nodes'. " r"\(choose from: id, u_rated\)",
+        match=r"Could not find attribute 'incorrect_attribute' for 'nodes'. \(choose from: id, u_rated\)",
     ):
         converter._convert_col_def_to_attribute(
             data=tabular_data_no_units_no_substitutions,

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/unit/converters/test_vision_excel_converter.py

@@ -146,4 +146,4 @@ def test_ambiguity_in_vision_excel():
     ambiguious_test_file = Path(__file__).parents[2] / "data" / "vision" / "excel_ambiguity_check_data.xlsx"
     excel_file_checker = ExcelAmbiguityChecker(file_path=ambiguious_test_file.as_posix())
     res, _ = excel_file_checker.check_ambiguity()
-    assert res == True
+    assert res

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/unit/data_stores/test_excel_file_store.py

@@ -4,7 +4,7 @@
 
 from pathlib import Path
 from typing import Dict
-from unittest.mock import MagicMock, call, mock_open, patch
+from unittest.mock import MagicMock, call, patch
 
 import numpy as np
 import pandas as pd

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/unit/mappings/test_field_mapping.py

@@ -3,7 +3,6 @@
 # SPDX-License-Identifier: MPL-2.0
 import logging
 
-import pytest
 import structlog
 
 from power_grid_model_io.mappings.field_mapping import FieldMapping

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/unit/utils/test_json.py

@@ -20,9 +20,8 @@ def test_compact_json_dump():
     string_stream = io.StringIO()
     compact_json_dump(data, string_stream, indent=2, max_level=0)
     assert (
-        string_stream.getvalue()
-        == """{"node": [{"id": 1, "x": 2}, {"id": 3, "x": 4}],"""
-        + """ "line": [{"id": 5, "x": 6}, {"id": 7, "x": {"y": 8.1, "z": 8.2}}]}"""
+        string_stream.getvalue() == """{"node": [{"id": 1, "x": 2}, {"id": 3, "x": 4}],"""
+        """ "line": [{"id": 5, "x": 6}, {"id": 7, "x": {"y": 8.1, "z": 8.2}}]}"""
     )
 
     string_stream = io.StringIO()
@@ -165,7 +164,7 @@ def test_json_encoder(numpy_type: type, native_type: type):
     value = encoder.default(value)
 
     # Assert
-    assert type(value) == native_type
+    assert type(value) is native_type
     assert value == 123
 
 
@@ -178,7 +177,7 @@ def test_json_encoder__np_array():
     value = encoder.default(value)
 
     # Assert
-    assert type(value) == list
+    assert type(value) is list
     assert value == [1, 2, 3]
 
 

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/utils.py

@@ -93,21 +93,20 @@ class MockFn:
             obj = copy(left)
         else:
             obj = MockFn(fn, left)
-        if MockFn._is_operator(right):
-            if (
-                obj.fn == "+"
-                and right.fn == "+"
-                or obj.fn == "-"
-                and right.fn == "+"
-                or obj.fn == "*"
-                and right.fn == "*"
-                or obj.fn == "/"
-                and right.fn == "*"
-                or obj.fn == "&"
-                and right.fn == "&"
-            ):
-                obj.args += right.args
-                return obj
+        if MockFn._is_operator(right) and (
+            obj.fn == "+"
+            and right.fn == "+"
+            or obj.fn == "-"
+            and right.fn == "+"
+            or obj.fn == "*"
+            and right.fn == "*"
+            or obj.fn == "/"
+            and right.fn == "*"
+            or obj.fn == "&"
+            and right.fn == "&"
+        ):
+            obj.args += right.args
+            return obj
         obj.args += [right]
         return obj
 
@@ -176,7 +175,7 @@ class MockFn:
                 return False
 
         def eq(left, right) -> bool:
-            if type(left) != type(right):
+            if type(left) is not type(right):
                 return False
             if isinstance(left, pd.DataFrame) and left.columns != right.columns:
                 return False

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/validation/conftest.py

@@ -2,15 +2,13 @@
 #
 # SPDX-License-Identifier: MPL-2.0
 
+from contextlib import suppress
+
 import pandas as pd
 
-try:
+with suppress(pd.errors.OptionError):
     pd.set_option("future.no_silent_downcasting", True)
-except pd.errors.OptionError:
-    pass
 
-try:
+with suppress(pd.errors.OptionError):
     # TODO(mgovers) We're ready for Pandas 3.x, but pandapower is not. Move to parent conftest when it is.
     pd.set_option("mode.copy_on_write", False)
-except pd.errors.OptionError:
-    pass

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/validation/converters/test_vision_excel_converter.py

@@ -95,7 +95,8 @@ def test_input_data(input_data: Tuple[SingleDataset, SingleDataset]):
 
 def test_input_data_custom_yaml():
     """
-    Unit test to preload the expected and actual data, using a different mapping file other than the one in the default location
+    Unit test to preload the expected and actual data, using a different
+    mapping file other than the one in the default location
     """
     for language in LANGUAGES:
         # Arrange

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/validation/utils.py

@@ -176,7 +176,7 @@ def load_json_single_dataset(file_path: Path, data_type: str) -> Tuple[SingleDat
     """
     try:
         dataset = json_deserialize_from_file(file_path=file_path)
-    except PowerGridSerializationError as error:
+    except PowerGridSerializationError:
         try:
             with warnings.catch_warnings():
                 warnings.simplefilter("ignore")
@@ -223,7 +223,7 @@ def compare_extra_info(actual: ExtraInfo, expected: ExtraInfo, component: str, o
             # If the values don't match, that's an error
             elif act[key] != value:
                 errors.append(
-                    f"Expected extra info '{key}' for {component} #{obj_id} to be {value}, " f"but it is {act[key]}."
+                    f"Expected extra info '{key}' for {component} #{obj_id} to be {value}, but it is {act[key]}."
                 )
 
     return errors

power_grid_model_io-1.3.5/PYPI_VERSION

@@ -1 +0,0 @@
-1.3.5

{power_grid_model_io-1.3.5 → power_grid_model_io-1.3.6}/tests/validation/converters/test_pandapower_converter_output.py

@@ -15,12 +15,12 @@ from power_grid_model.validation import assert_valid_input_data
 from power_grid_model_io.converters import PandaPowerConverter
 from power_grid_model_io.converters.pandapower_converter import PandaPowerData
 
-pp = pytest.importorskip("pandapower", reason="pandapower is not installed")
-# we add this to enable python 3.13 testing even though pandapower 3.0 is not yet compatible with it
-
 from ...data.pandapower.pp_validation import pp_net, pp_net_3ph
 from ..utils import component_attributes_df, load_json_single_dataset
 
+pp = pytest.importorskip("pandapower", reason="pandapower is not installed")
+# we add this to enable python 3.13 testing even though pandapower 3.0 is not yet compatible with it
+
 PGM_PP_TEST_DATA = Path(__file__).parents[2] / "data" / "pandapower"
 PGM_OUTPUT_FILE = PGM_PP_TEST_DATA / "pgm_output_data.json"
 PGM_ASYM_OUTPUT_FILE = PGM_PP_TEST_DATA / "pgm_asym_output_data.json"