Merge pull request #4562 from GitPaean/small_cleaning_up

removing something not used or not defined even

Author: bska

opm/input/eclipse/Schedule/Group/GuideRate.cpp

@@ -358,16 +358,6 @@ double Opm::GuideRate::eval_form(const GuideRateModel& model,
     return model.eval(oil_pot, gas_pot, wat_pot);
 }
 
-double Opm::GuideRate::eval_group_pot() const
-{
-    return 0.0;
-}
-
-double Opm::GuideRate::eval_group_resvinj() const
-{
-    return 0.0;
-}
-
 void Opm::GuideRate::assign_grvalue(const std::string&    wgname,
                                     const GuideRateModel& model,
                                     GuideRateValue&&      value)

opm/input/eclipse/Schedule/Group/GuideRate.hpp

@@ -206,8 +206,6 @@ class GuideRate
                      const double          oil_pot,
                      const double          gas_pot,
                      const double          wat_pot) const;
-    double eval_group_pot() const;
-    double eval_group_resvinj() const;
 
     void assign_grvalue(const std::string&    wgname,
                         const GuideRateModel& model,

opm/input/eclipse/Schedule/Schedule.cpp

@@ -2180,11 +2180,6 @@ File {} line {}.)", pattern, location.keyword, location.filename, location.linen
         return fmt::format("{:04d}-{:02d}-{:02d}" , ts.year(), ts.month(), ts.day());
     }
 
-    std::string Schedule::simulationDays(std::size_t currentStep) const {
-        const double sim_time { this->m_static.m_unit_system.from_si(UnitSystem::measure::time, simTime(currentStep)) } ;
-        return fmt::format("{} {}", sim_time, this->m_static.m_unit_system.name(UnitSystem::measure::time));
-    }
-
 namespace {
 
     // Duplicated from Well.cpp

opm/input/eclipse/Schedule/Schedule.hpp

@@ -325,7 +325,6 @@ namespace Opm {
         WellProducerCMode getGlobalWhistctlMmode(std::size_t timestep) const;
 
         const UDQConfig& getUDQConfig(std::size_t timeStep) const;
-        void evalAction(const SummaryState& summary_state, std::size_t timeStep);
 
         GTNode groupTree(std::size_t report_step) const;
         GTNode groupTree(const std::string& root_node, std::size_t report_step) const;
@@ -526,7 +525,6 @@ namespace Opm {
 
         void updateGuideRateModel(const GuideRateModel& new_model, std::size_t report_step);
         GTNode groupTree(const std::string& root_node, std::size_t report_step, std::size_t level, const std::optional<std::string>& parent_name) const;
-        bool checkGroups(const ParseContext& parseContext, ErrorGuard& errors);
         bool updateWellStatus( const std::string& well, std::size_t reportStep, WellStatus status, std::optional<KeywordLocation> = {});
         void addWellToGroup( const std::string& group_name, const std::string& well_name , std::size_t timeStep);
         void iterateScheduleSection(std::size_t load_start,
@@ -570,9 +568,7 @@ namespace Opm {
         std::vector<std::string> wellNames(const std::string& pattern,
                                            const HandlerContext& context,
                                            bool allowEmpty = false);
-        std::vector<std::string> wellNames(const std::string& pattern, std::size_t timeStep, const std::vector<std::string>& matching_wells, InputErrorAction error_action, ErrorGuard& errors, const KeywordLocation& location) const;
         static std::string formatDate(std::time_t t);
-        std::string simulationDays(std::size_t currentStep) const;
         void applyGlobalWPIMULT( const std::unordered_map<std::string, double>& wpimult_global_factor);
 
         bool must_write_rst_file(std::size_t report_step) const;

opm/input/eclipse/Schedule/Well/WellTestConfig.cpp

@@ -162,25 +162,6 @@ const WellTestConfig::WTESTWell& WellTestConfig::get(const std::string& well) co
 }
 
 
-
-std::string WellTestConfig::reasonToString(const Reason reason) {
-    switch(reason) {
-    case Reason::PHYSICAL:
-        return std::string("PHYSICAL");
-    case Reason::ECONOMIC:
-        return std::string("ECONOMIC");
-    case Reason::GROUP:
-        return std::string("GROUP");
-    case Reason::THP_DESIGN:
-        return std::string("THP_DESIGN");
-    case Reason::COMPLETION:
-        return std::string("COMPLETION");
-    default:
-        throw std::runtime_error("unknown closure reason");
-    }
-}
-
-
 bool WellTestConfig::empty() const {
     return this->wells.empty();
 }

opm/input/eclipse/Schedule/Well/WellTestConfig.hpp

@@ -120,7 +120,6 @@ class WellTestConfig {
     bool has(const std::string& well, Reason reason) const;
     const WTESTWell& get(const std::string& well) const;
 
-    static std::string reasonToString(const Reason reason);
     bool empty() const;
 
     bool operator==(const WellTestConfig& data) const;