fix some issue with pandapower when loads / gens / storages were disconnected

Signed-off-by: DONNOT Benjamin <[email protected]>

Author: BDonnot

.github/workflows/main.yml

@@ -52,7 +52,7 @@ jobs:
     steps:
 
       - name: Checkout sources
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           submodules: true
 
@@ -134,7 +134,7 @@ jobs:
     steps:
 
       - name: Checkout sources
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           submodules: true
 
@@ -212,7 +212,7 @@ jobs:
     steps:
 
       - name: Checkout sources
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           submodules: true
 

CHANGELOG.rst

@@ -188,6 +188,11 @@ Native multi agents support:
   it... unless you know what you are doing)
 - [IMPROVED] `ForecastEnv` is now part of the public API.
 
+[1.10.5] - 2025-03-07
+------------------------
+- [FIXED] force pandapower < 3 otherwise pandapower backend does not work and 
+  lots of tests are failing.
+
 [1.10.4] - 2024-10-15
 -------------------------
 - [FIXED] new pypi link (no change in code)

docs/conf.py

@@ -18,7 +18,7 @@
 # -- Project information -----------------------------------------------------
 
 project = 'Grid2Op'
-copyright = '2019, RTE France'
+copyright = 'Grid2Op a Series of LF Projects, LLC,\nFor website terms of use, trademark policy and other project policies please see https://lfprojects.org.'
 author = 'Benjamin Donnot'
 
 # The full version, including alpha/beta/rc tags

docs/final.rst

@@ -6,3 +6,6 @@ If you still can't find what you're looking for, try in one of the following pag
 
 Still trouble finding the information ? Do not hesitate to send a github issue about the documentation at this
 link: `Documentation issue template <https://github.com/Grid2Op/grid2op/issues/new?assignees=&labels=documentation&template=documentation.md&title=>`_
+
+Copyright © Grid2Op a Series of LF Projects, LLC
+For website terms of use, trademark policy and other project policies please see https://lfprojects.org.

grid2op/Backend/pandaPowerBackend.py

@@ -1495,6 +1495,9 @@ def _loads_info(self):
         load_theta = self._grid.res_bus.loc[self._grid.load["bus"].values][
             "va_degree"
         ].values.astype(dt_float)
+        load_disco = ~self._grid.load["in_service"]
+        load_v[load_disco] = 0.
+        load_theta[load_disco] = 0.
         return load_p, load_q, load_v, load_theta
 
     def generators_info(self) -> Tuple[np.ndarray, np.ndarray, np.ndarray]:

grid2op/tests/test_detached_properly_updated.py

@@ -0,0 +1,128 @@
+# Copyright (c) 2023, RTE (https://www.rte-france.com)
+# See AUTHORS.txt
+# This Source Code Form is subject to the terms of the Mozilla Public License, version 2.0.
+# If a copy of the Mozilla Public License, version 2.0 was not distributed with this file,
+# you can obtain one at http://mozilla.org/MPL/2.0/.
+# SPDX-License-Identifier: MPL-2.0
+# This file is part of LightSim2grid, LightSim2grid implements a c++ backend targeting the Grid2Op platform.
+
+import unittest
+import warnings
+
+import grid2op
+from grid2op.Action import CompleteAction
+
+
+class DetachmentBackendOutputTester(unittest.TestCase):
+    """issue is still not replicated and these tests pass"""
+    def setUp(self) -> None:
+        with warnings.catch_warnings():
+            warnings.filterwarnings("ignore")
+            self.env = grid2op.make("educ_case14_storage", 
+                                     test=True,
+                                     allow_detachment=True,
+                                     action_class=CompleteAction)
+        self.env.reset(seed=0, options={"time serie id": 0})
+        obs = self.env.reset()
+        return super().setUp()
+    
+    def tearDown(self) -> None:
+        self.env.close()
+        return super().tearDown()
+    
+    def test_disco_gen(self):
+        """test i can disconnect a gen"""
+        gen_id = 0
+        obs, reward, done, info = self.env.step(self.env.action_space(
+            {
+                "set_bus": {"generators_id": [(gen_id, -1)]}
+            }
+        ))
+        assert not done
+        assert obs.gen_detached[gen_id]
+        assert obs.gen_p[gen_id] == 0.
+        assert obs.gen_v[gen_id] == 0.
+        assert obs.gen_q[gen_id] == 0.
+        assert obs.gen_theta[gen_id] == 0., f"{obs.gen_theta[gen_id]} vs 0."
+        assert abs(obs.gen_p_detached[gen_id] - 79.8) <= 1e-5
+        
+        obs1, _, done, info = self.env.step(self.env.action_space({}))
+        assert not done
+        assert obs.gen_detached[gen_id]
+        assert obs1.gen_p[gen_id] == 0.
+        assert obs1.gen_v[gen_id] == 0.
+        assert obs1.gen_q[gen_id] == 0.
+        assert obs1.gen_theta[gen_id] == 0., f"{obs1.gen_theta[gen_id]} vs 0."
+        assert abs(obs1.gen_p_detached[gen_id] - 80.5) <= 1e-5
+        
+        obs2, _, done, info = self.env.step(self.env.action_space({}))
+        assert not done
+        assert obs.gen_detached[gen_id]
+        assert obs2.gen_p[gen_id] == 0.
+        assert obs2.gen_v[gen_id] == 0.
+        assert obs2.gen_q[gen_id] == 0.
+        assert obs2.gen_theta[gen_id] == 0., f"{obs2.gen_theta[gen_id]} vs 0."
+        assert abs(obs2.gen_p_detached[gen_id] - 80.5) <= 1e-5
+        
+    def test_disco_load(self):
+        """test i can disconnect a load"""
+        load_id = 0
+        obs, reward, done, info = self.env.step(self.env.action_space(
+            {
+                "set_bus": {"loads_id": [(load_id, -1)]}
+            }
+        ))
+        assert not done
+        assert obs.load_detached[load_id]
+        assert obs.load_p[load_id] == 0., f"{obs.load_p[load_id]} vs 0."
+        assert obs.load_q[load_id] == 0., f"{obs.load_q[load_id]} vs 0."
+        assert obs.load_v[load_id] == 0., f"{obs.load_v[load_id]} vs 0."
+        assert obs.load_theta[load_id] == 0., f"{obs.load_theta[load_id]} vs 0."
+        assert abs(obs.load_p_detached[load_id] - 21.9) <= 1e-5
+        
+        obs1, _, done, info = self.env.step(self.env.action_space({}))
+        assert not done
+        assert obs1.load_detached[load_id]
+        assert obs1.load_p[load_id] == 0., f"{obs1.load_p[load_id]} vs 0."
+        assert obs1.load_q[load_id] == 0., f"{obs1.load_q[load_id]} vs 0."
+        assert obs1.load_v[load_id] == 0., f"{obs1.load_v[load_id]} vs 0."
+        assert obs1.load_theta[load_id] == 0., f"{obs1.load_theta[load_id]} vs 0."
+        assert abs(obs1.load_p_detached[load_id] - 21.7) <= 1e-5
+        
+        obs2, _, done, info = self.env.step(self.env.action_space({}))
+        assert not done
+        assert obs2.load_detached[load_id]
+        assert obs2.load_p[load_id] == 0., f"{obs2.load_p[load_id]} vs 0."
+        assert obs2.load_q[load_id] == 0., f"{obs2.load_q[load_id]} vs 0."
+        assert obs2.load_v[load_id] == 0., f"{obs2.load_v[load_id]} vs 0."
+        assert obs2.load_theta[load_id] == 0., f"{obs2.load_theta[load_id]} vs 0."
+        assert abs(obs2.load_p_detached[load_id] - 21.6) <= 1e-5
+    
+    def test_disco_storage(self):
+        """test i can disconnect a storage unit"""
+        sto_id = 0
+        obs, reward, done, info = self.env.step(self.env.action_space(
+            {
+                "set_bus": {"storages_id": [(sto_id, -1)]},
+                "set_storage": [(0, 1.)]
+            }
+        ))
+        assert not done
+        assert obs.storage_detached[sto_id]
+        assert obs.storage_power[sto_id] == 0., f"{obs.storage_power[sto_id]} vs 0."
+        assert abs(obs.storage_p_detached[sto_id] - 1.) <= 1e-5, f"{obs.storage_p_detached[sto_id]} vs 1."
+        
+        obs1, _, done, info = self.env.step(self.env.action_space({}))
+        assert not done
+        assert obs1.storage_detached[sto_id]
+        assert obs1.storage_power[sto_id] == 0., f"{obs1.storage_power[sto_id]} vs 0."
+        
+        obs2, _, done, info = self.env.step(self.env.action_space({}))
+        assert not done
+        assert obs2.storage_detached[sto_id]
+        assert obs2.storage_power[sto_id] == 0., f"{obs2.storage_power[sto_id]} vs 0."
+        
+        
+if __name__ == "__main__":
+    unittest.main()
+