Skip to content

Commit 53f80f4

Browse files
committed
cleanup
Signed-off-by: Martijn Govers <[email protected]>
1 parent b60c807 commit 53f80f4

File tree

6 files changed

+10
-29
lines changed

6 files changed

+10
-29
lines changed

power_grid_model_c/power_grid_model/include/power_grid_model/common/calculation_info.hpp

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010

1111
namespace power_grid_model {
1212
namespace common::logging {
13-
class CalculationInfo final : public Logger {
13+
class CalculationInfo : public Logger {
1414
using Data = std::map<LogEvent, double>;
1515

1616
public:
@@ -87,6 +87,13 @@ class CalculationInfo final : public Logger {
8787
void clear() { data_.clear(); }
8888
};
8989

90+
inline Logger& merge_into(Logger& destination, CalculationInfo const& source) {
91+
for (const auto& [tag, value] : source.report()) {
92+
destination.log(tag, value);
93+
}
94+
return destination;
95+
}
96+
9097
class MultiThreadedCalculationInfo : public MultiThreadedLoggerImpl<CalculationInfo> {
9198
public:
9299
using MultiThreadedLoggerImpl<CalculationInfo>::MultiThreadedLoggerImpl;

power_grid_model_c/power_grid_model/include/power_grid_model/job_adapter.hpp

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,8 @@
1010
#include "job_interface.hpp"
1111
#include "main_model_fwd.hpp"
1212

13-
#include "main_core/calculation_info.hpp"
1413
#include "main_core/update.hpp"
1514

16-
#include <mutex>
17-
1815
namespace power_grid_model {
1916

2017
template <class MainModel, typename... ComponentType> class JobAdapter;
@@ -100,8 +97,6 @@ class JobAdapter<MainModel, ComponentList<ComponentType...>>
10097

10198
Logger* log_{nullptr};
10299

103-
std::mutex calculation_info_mutex_;
104-
105100
void calculate_impl(MutableDataset const& result_data, Idx scenario_idx) const {
106101
MainModel::calculator(options_.get(), model_reference_.get(), result_data.get_individual_scenario(scenario_idx),
107102
false);

power_grid_model_c/power_grid_model/include/power_grid_model/job_dispatch.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,8 @@
88
#include "job_interface.hpp"
99

1010
#include "common/exception.hpp"
11+
#include "common/timer.hpp"
1112
#include "common/typing.hpp"
12-
#include "main_core/calculation_info.hpp"
1313

1414
#include <thread>
1515

power_grid_model_c/power_grid_model/include/power_grid_model/main_core/calculation_info.hpp

Lines changed: 0 additions & 20 deletions
This file was deleted.

power_grid_model_c/power_grid_model/include/power_grid_model/main_model_impl.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
#include "optimizer/optimizer.hpp"
3131

3232
// main model implementation
33-
#include "main_core/calculation_info.hpp"
3433
#include "main_core/calculation_input_preparation.hpp"
3534
#include "main_core/core_utils.hpp"
3635
#include "main_core/input.hpp"

tests/benchmark_cpp/benchmark.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,7 @@ struct PowerGridBenchmark {
152152
try {
153153
// calculate
154154
main_model->calculate(model_options, output.get_dataset(), batch_data.get_dataset());
155-
main_core::merge_into(info, main_model->calculation_info());
155+
common::logging::merge_into(info, main_model->calculation_info());
156156
} catch (std::exception const& e) {
157157
std::cout << std::format("\nAn exception was raised during execution: {}\n", e.what());
158158
}

0 commit comments

Comments
 (0)