From 1e6505909ac218dea20c89e524e663b062bf8b77 Mon Sep 17 00:00:00 2001 From: Abdoulbari ZAKIR Date: Wed, 6 Sep 2023 16:23:55 +0200 Subject: [PATCH] more on major code smells --- src/solver/optimisation/MinDownTime.cpp | 8 --- src/solver/optimisation/MinHydroPower.cpp | 2 - src/solver/optimisation/MinHydroPower.h | 3 +- .../NbUnitsOutageLessThanNbUnitsStop.cpp | 14 ----- .../PMaxDispatchableGeneration.cpp | 2 - .../PMinDispatchableGeneration.cpp | 4 +- src/solver/optimisation/constraint_builder.h | 57 ------------------- ...nstruction_contraintes_couts_demarrage.cpp | 6 -- ...n_matrice_des_contraintes_cas_lineaire.cpp | 1 - 9 files changed, 3 insertions(+), 94 deletions(-) diff --git a/src/solver/optimisation/MinDownTime.cpp b/src/solver/optimisation/MinDownTime.cpp index 152958ea59..170c4179e0 100644 --- a/src/solver/optimisation/MinDownTime.cpp +++ b/src/solver/optimisation/MinDownTime.cpp @@ -12,9 +12,6 @@ void MinDownTime::add(int pays, int cluster, int clusterIndex, int pdt, bool Sim CorrespondanceCntNativesCntOptim.NumeroDeContrainteDesContraintesDeDureeMinDArret[cluster] = -1; if (!Simulation) { - double pminDUnGroupeDuPalierThermique - = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[clusterIndex]; - int NombreDePasDeTempsPourUneOptimisation = problemeHebdo->NombreDePasDeTempsPourUneOptimisation; @@ -22,11 +19,6 @@ void MinDownTime::add(int pays, int cluster, int clusterIndex, int pdt, bool Sim = PaliersThermiquesDuPays.PuissanceDisponibleEtCout[clusterIndex] .NombreMaxDeGroupesEnMarcheDuPalierThermique; - int t1 = pdt - DureeMinimaleDArretDUnGroupeDuPalierThermique; - if (t1 < 0) - t1 = NombreDePasDeTempsPourUneOptimisation + t1; - double rhs = NombreMaxDeGroupesEnMarcheDuPalierThermique[t1]; // /!\ TODO check - builder.updateHourWithinWeek(pdt).include(Variable::NODU(cluster), 1.0); for (int k = pdt - DureeMinimaleDArretDUnGroupeDuPalierThermique + 1; k <= pdt; k++) diff --git a/src/solver/optimisation/MinHydroPower.cpp b/src/solver/optimisation/MinHydroPower.cpp index a3472397e4..28ba696ec5 100644 --- a/src/solver/optimisation/MinHydroPower.cpp +++ b/src/solver/optimisation/MinHydroPower.cpp @@ -25,8 +25,6 @@ void MinHydroPower::add(int pays) builder.updateHourWithinWeek(pdt); builder.include(Variable::HydProd(pays), 1.0); } - // const double rhs = problemeHebdo->CaracteristiquesHydrauliques[pays] - // .MinEnergieHydrauParIntervalleOptimise[NumeroDeLIntervalle]; problemeHebdo->NumeroDeContrainteMinEnergieHydraulique[pays] = problemeHebdo->ProblemeAResoudre->NombreDeContraintes; diff --git a/src/solver/optimisation/MinHydroPower.h b/src/solver/optimisation/MinHydroPower.h index b36b9dbeba..6423cbcdd7 100644 --- a/src/solver/optimisation/MinHydroPower.h +++ b/src/solver/optimisation/MinHydroPower.h @@ -1,8 +1,9 @@ #pragma once #include "constraint_builder.h" -struct MinHydroPower : private Constraint +class MinHydroPower : private Constraint { +public: using Constraint::Constraint; void add(int pays); }; diff --git a/src/solver/optimisation/NbUnitsOutageLessThanNbUnitsStop.cpp b/src/solver/optimisation/NbUnitsOutageLessThanNbUnitsStop.cpp index 8efb687f22..dee28fa384 100644 --- a/src/solver/optimisation/NbUnitsOutageLessThanNbUnitsStop.cpp +++ b/src/solver/optimisation/NbUnitsOutageLessThanNbUnitsStop.cpp @@ -10,8 +10,6 @@ void NbUnitsOutageLessThanNbUnitsStop::add(int pays, { const PALIERS_THERMIQUES& PaliersThermiquesDuPays = problemeHebdo->PaliersThermiquesDuPays[pays]; - double pminDUnGroupeDuPalierThermique - = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[clusterIndex]; const int DureeMinimaleDArretDUnGroupeDuPalierThermique = PaliersThermiquesDuPays.DureeMinimaleDArretDUnGroupeDuPalierThermique[clusterIndex]; @@ -20,18 +18,6 @@ void NbUnitsOutageLessThanNbUnitsStop::add(int pays, CorrespondanceCntNativesCntOptim.NumeroDeContrainteDesContraintesDeDureeMinDeMarche[cluster] = -1; - int NombreDePasDeTempsPourUneOptimisation - = problemeHebdo->NombreDePasDeTempsPourUneOptimisation; - - int t1 = pdt - DureeMinimaleDArretDUnGroupeDuPalierThermique; - - if (t1 < 0) - t1 = NombreDePasDeTempsPourUneOptimisation + t1; - - const std::vector& NombreMaxDeGroupesEnMarcheDuPalierThermique - = PaliersThermiquesDuPays.PuissanceDisponibleEtCout[clusterIndex] - .NombreMaxDeGroupesEnMarcheDuPalierThermique; - builder.updateHourWithinWeek(pdt) .include(Variable::NumberBreakingDownDispatchableUnits(cluster), 1.0) .include(Variable::NumberStoppingDispatchableUnits(cluster), -1.0) diff --git a/src/solver/optimisation/PMaxDispatchableGeneration.cpp b/src/solver/optimisation/PMaxDispatchableGeneration.cpp index 854d3a100a..8dd5d50f25 100644 --- a/src/solver/optimisation/PMaxDispatchableGeneration.cpp +++ b/src/solver/optimisation/PMaxDispatchableGeneration.cpp @@ -12,8 +12,6 @@ void PMaxDispatchableGeneration::add(int pays, = problemeHebdo->PaliersThermiquesDuPays[pays]; double pmaxDUnGroupeDuPalierThermique = PaliersThermiquesDuPays.PmaxDUnGroupeDuPalierThermique[clusterIndex]; - const int DureeMinimaleDArretDUnGroupeDuPalierThermique - = PaliersThermiquesDuPays.DureeMinimaleDArretDUnGroupeDuPalierThermique[clusterIndex]; builder.updateHourWithinWeek(pdt) .include(Variable::DispatchableProduction(cluster), 1.0) diff --git a/src/solver/optimisation/PMinDispatchableGeneration.cpp b/src/solver/optimisation/PMinDispatchableGeneration.cpp index 00d5f2adcf..2d0595e67b 100644 --- a/src/solver/optimisation/PMinDispatchableGeneration.cpp +++ b/src/solver/optimisation/PMinDispatchableGeneration.cpp @@ -13,13 +13,11 @@ void PMinDispatchableGeneration::add(int pays, double pminDUnGroupeDuPalierThermique = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[clusterIndex]; - int NombreDePasDeTempsPourUneOptimisation - = problemeHebdo->NombreDePasDeTempsPourUneOptimisation; builder.updateHourWithinWeek(pdt) .include(Variable::DispatchableProduction(cluster), 1.0) .include(Variable::NODU(cluster), -pminDUnGroupeDuPalierThermique) .greaterThan(); - /*consider Adding rename constraint inside the builder*/ + /*consider Adding naming constraint inside the builder*/ if (builder.NumberOfVariables() > 0) { ConstraintNamer namer(problemeHebdo->ProblemeAResoudre->NomDesContraintes, diff --git a/src/solver/optimisation/constraint_builder.h b/src/solver/optimisation/constraint_builder.h index 482f62d531..c912e99061 100644 --- a/src/solver/optimisation/constraint_builder.h +++ b/src/solver/optimisation/constraint_builder.h @@ -242,32 +242,6 @@ class Visitor }; } // namespace Variable -// enum class Constraint { -// FlowDissociation, -// AreaBalance, -// FictiveLoads, -// HydroPower, -// HydroPowerSmoothingUsingVariationSum, -// HydroPowerSmoothingUsingVariationMaxDown, -// HydroPowerSmoothingUsingVariationMaxUp, -// MinHydroPower, -// MaxHydroPower, -// MaxPumping, -// AreaHydroLevel, -// FinalStockEquivalent, -// FinalStockExpression, -// NbUnitsOutageLessThanNbUnitsStop, -// NbDispUnitsMinBoundSinceMinUpTime, -// MinDownTime, -// PMaxDispatchableGeneration, -// PMinDispatchableGeneration, -// ConsistenceNODU, -// ShortTermStorageLevel, -// BindingConstraintHour, -// BindingConstraintDay, -// BindingConstraintWeek -// }; - class ConstraintBuilder { public: @@ -307,18 +281,6 @@ class ConstraintBuilder return *this; } - // ConstraintBuilder& operatorRHS(char op, double rhs) - // { - // if (op == '<' || op == '=' || op == '>') - // { - // operator_ = op; - // rhs_ = rhs; - // } - // else - // throw std::runtime_error("Invalid operator"); - - // return *this; - // } ConstraintBuilder& equalTo() { @@ -336,25 +298,6 @@ class ConstraintBuilder return *this; } - // ConstraintBuilder& equalTo(double rhs) - // { - // operator_ = '='; - // rhs_ = rhs; - // return *this; - // } - // ConstraintBuilder& lessThan(double rhs) - // { - // operator_ = '<'; - // rhs_ = rhs; - // return *this; - // } - // ConstraintBuilder& greaterThan(double rhs) - // { - // operator_ = '>'; - // rhs_ = rhs; - // return *this; - // } - void build(); int NumberOfVariables() const { diff --git a/src/solver/optimisation/opt_construction_contraintes_couts_demarrage.cpp b/src/solver/optimisation/opt_construction_contraintes_couts_demarrage.cpp index 8fab4458c5..9945b8e909 100644 --- a/src/solver/optimisation/opt_construction_contraintes_couts_demarrage.cpp +++ b/src/solver/optimisation/opt_construction_contraintes_couts_demarrage.cpp @@ -64,10 +64,6 @@ void OPT_ConstruireLaMatriceDesContraintesDuProblemeLineaireCoutsDeDemarrage( { PMaxDispatchableGeneration pMaxDispatchableGeneration(problemeHebdo); PMinDispatchableGeneration pMinDispatchableGeneration(problemeHebdo); - double pminDUnGroupeDuPalierThermique - = PaliersThermiquesDuPays.pminDUnGroupeDuPalierThermique[index]; - double pmaxDUnGroupeDuPalierThermique - = PaliersThermiquesDuPays.PmaxDUnGroupeDuPalierThermique[index]; const int palier = PaliersThermiquesDuPays.NumeroDuPalierDansLEnsembleDesPaliersThermiques[index]; @@ -156,8 +152,6 @@ void OPT_ConstruireLaMatriceDesContraintesDuProblemeLineaireCoutsDeDemarrage( MinDownTime minDownTime(problemeHebdo); for (int index = 0; index < PaliersThermiquesDuPays.NombreDePaliersThermiques; index++) { - int DureeMinimaleDArretDUnGroupeDuPalierThermique - = PaliersThermiquesDuPays.DureeMinimaleDArretDUnGroupeDuPalierThermique[index]; const int palier = PaliersThermiquesDuPays.NumeroDuPalierDansLEnsembleDesPaliersThermiques[index]; diff --git a/src/solver/optimisation/opt_construction_matrice_des_contraintes_cas_lineaire.cpp b/src/solver/optimisation/opt_construction_matrice_des_contraintes_cas_lineaire.cpp index 74cd53e4f0..345bb53ddf 100644 --- a/src/solver/optimisation/opt_construction_matrice_des_contraintes_cas_lineaire.cpp +++ b/src/solver/optimisation/opt_construction_matrice_des_contraintes_cas_lineaire.cpp @@ -133,7 +133,6 @@ void OPT_ConstruireLaMatriceDesContraintesDuProblemeLineaire(PROBLEME_HEBDO* pro if (nombreDePasDeTempsPourUneOptimisation > nombreDePasDeTempsDUneJournee) { - int semaine = problemeHebdo->weekInTheYear; CORRESPONDANCES_DES_CONTRAINTES_HEBDOMADAIRES& CorrespondanceCntNativesCntOptimHebdomadaires = problemeHebdo->CorrespondanceCntNativesCntOptimHebdomadaires; for (uint32_t cntCouplante = 0; cntCouplante < problemeHebdo->NombreDeContraintesCouplantes;