Skip to content

Commit 5b6113f

Browse files
committed
assertion in combine magnitude causes crashes in first batch validation trial run
Signed-off-by: Martijn Govers <Martijn.Govers@Alliander.com>
1 parent 51d0e8e commit 5b6113f

File tree

1 file changed

+0
-8
lines changed
  • power_grid_model_c/power_grid_model/include/power_grid_model/common

1 file changed

+0
-8
lines changed

power_grid_model_c/power_grid_model/include/power_grid_model/common/statistics.hpp

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -274,14 +274,6 @@ template <std::ranges::view RandVarsView>
274274
constexpr auto combine_magnitude(RandVarsView rand_vars) {
275275
using sym = std::ranges::range_value_t<RandVarsView>::sym;
276276

277-
assert(std::ranges::all_of(rand_vars, [](auto const& measurement) {
278-
if constexpr (is_symmetric_v<sym>) {
279-
return (!is_nan(imag(measurement.value))) || real(measurement.value) >= 0.0;
280-
} else {
281-
return (!is_nan(imag(measurement.value))) || (real(measurement.value) >= 0.0).all();
282-
}
283-
}));
284-
285277
auto const weighted_average_magnitude_measurement =
286278
statistics::combine(rand_vars | std::views::transform([](auto const& measurement) {
287279
return UniformRealRandVar<sym>{.value = detail::cabs_or_real<sym>(measurement.value),

0 commit comments

Comments
 (0)