Martijn Govers

Results 24 issues of Martijn Govers

**Describe the bug** according to the Clang static analyzer, `msgpack::zone` calls `::free` on memory that has been cleaned up before: https://clang.llvm.org/docs/analyzer/checkers.html#cplusplus-newdelete Verified to be in `msgpack-cxx` version `6.1.0`, but likely...

**Issue** Platform- and environment-independently downloading the latest release in automation software is flaky. **Background** In our workflow, we would like to build a Python wrapper around a C library. If...

This makes the C++ API more easy to develop/use Blocks #597

improvement

* Add automatic tap position as experimental feature to C API * Add `PGM_TapChangingStrategy` enum * Add `PGM_set_tap_changing_strategy` option setter

feature

During implementation of #588 , it was discovered that, when line drop compensation is large, there are cases where the assumption that the voltage on the controlled side decreases with...

feature

In #586 , it was discovered that, when a small voltage band is provided, the Tap Position Optimizer starts oscillating indefinitely. This is captured by the code in https://github.com/PowerGridModel/power-grid-model/pull/586/files#diff-e3cb2a7b8db22693560826087f2c675630f9cd51a2337191aadc986dd50c4249R803-R812 ,...

bug

**Describe the feature request** Since `C++20`, the `` STL functionality was introduced to improve string formatting: * more readable * less use of `std::to_string` (only when strictly necessary) * no...

feature
good first issue

The [`clang-tidy` action](https://github.com/PowerGridModel/power-grid-model/actions/workflows/clang-tidy.yml) checks for static analyzer warnings (similarly to SonarCube). Currently, there are quite some checks that are disabled in the [.clang-tidy](https://github.com/PowerGridModel/power-grid-model/blob/main/.clang-tidy) file. Disabled warnings are prefixed `-` in...

feature

Fixes #433

improvement

## Describe the feature request Expose all different error types thrown by the calculation core as separate error classes that derive from `PowerGridError`. ## Rationale Currently, all power grid errors...

feature