From ffafd01757197350b21928eead0c347bab5032b8 Mon Sep 17 00:00:00 2001
From: "github-actions[bot]" <github-actions[bot]@users.noreply.github.com>
Date: Sat, 28 Sep 2024 07:55:27 +0800
Subject: [PATCH] =?UTF-8?q?01.=20Fix=20missing=20dt=20multiplication=20in?=
 =?UTF-8?q?=20ramping=20and=20co2=20emission=20calculation=20when=20dt=20?=
 =?UTF-8?q?=E2=89=A0=201;=2002.=20Fix=20repeated=20dt=20multiplication=20i?=
 =?UTF-8?q?n=20generation=20cost=20calculation?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 prepshot/_model/co2.py        | 4 ++--
 prepshot/_model/cost.py       | 9 +++------
 prepshot/_model/generation.py | 5 +++--
 3 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/prepshot/_model/co2.py b/prepshot/_model/co2.py
index bb1ec05..a7e6570 100644
--- a/prepshot/_model/co2.py
+++ b/prepshot/_model/co2.py
@@ -134,5 +134,5 @@ def carbon_breakdown(
         """
         model = self.model
         ef = model.params['emission_factor'][te, y]
-        dt = model.params['dt']
-        return ef * dt * poi.quicksum(model.gen.select('*', '*', y, z, te))
+        w = model.params['weight']
+        return 1 / w * ef * poi.quicksum(model.gen.select('*', '*', y, z, te))
diff --git a/prepshot/_model/cost.py b/prepshot/_model/cost.py
index 585c93a..50aea5f 100644
--- a/prepshot/_model/cost.py
+++ b/prepshot/_model/cost.py
@@ -101,10 +101,9 @@ def fuel_cost_breakdown(
         """
         model = self.model
         fp = model.params['fuel_price'][te, y]
-        dt = model.params['dt']
         vf = model.params['var_factor'][y]
         w = model.params['weight']
-        return (1 / w * fp * dt * vf
+        return (1 / w * fp * vf
             * poi.quicksum(model.gen.select('*', '*', y, z, te)))
 
     def cost_var_line_breakdown(
@@ -130,10 +129,9 @@ def cost_var_line_breakdown(
         """
         model = self.model
         lvc = model.params['transmission_line_variable_OM_cost'][z, z1]
-        dt = model.params['dt']
         vf = model.params['var_factor'][y]
         w = model.params['weight']
-        return (0.5 / w * lvc * dt * vf
+        return (0.5 / w * lvc * vf
             * poi.quicksum(model.trans_export.select('*', '*', y, z, z1)))
 
     def cost_var_tech_breakdown(
@@ -158,10 +156,9 @@ def cost_var_tech_breakdown(
         """
         model = self.model
         tvc = model.params['technology_variable_OM_cost'][te, y]
-        dt = model.params['dt']
         vf = model.params['var_factor'][y]
         w = model.params['weight']
-        return (1 / w * tvc * dt * vf
+        return (1 / w * tvc * vf
             * poi.quicksum(model.gen.select('*', '*', y, z, te)))
 
     def cost_fix_line_breakdown(
diff --git a/prepshot/_model/generation.py b/prepshot/_model/generation.py
index 097c724..92d756b 100644
--- a/prepshot/_model/generation.py
+++ b/prepshot/_model/generation.py
@@ -163,11 +163,12 @@ def ramping_up_rule(
             The constraint of the model.
         """
         model = self.model
-        rp = model.params['ramp_up'][te] * model.params['dt']
+        dt = model.params['dt']
+        rp = model.params['ramp_up'][te] * dt
         if rp < 1 < h:
             lhs = (
                 model.gen[h, m, y, z, te] - model.gen[h-1, m, y, z, te]
-                - rp * model.cap_existing[y, z, te]
+                - rp * model.cap_existing[y, z, te] * dt
             )
             return model.add_linear_constraint(lhs, poi.Leq, 0)