Mcmc config by francoisbonneau · Pull Request #39 · Geode-solutions/OpenGeode-Stochastic

francoisbonneau · 2026-05-25T11:40:50Z

No description provided.

feat(Model): create a factory for energy terms.

feat(EnergyTerm): builder for energy terms

github-actions · 2026-05-25T11:49:45Z

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v20.1.8) reports: 449 concern(s)

bindings/python/src/stochastic/sampling/mcmc/helpers/fracture_simulation_runner.hpp:30:9: warning: [google-build-using-namespace]

do not use namespace using-directives; use using-declarations instead
```
   30 |         using namespace geode;
      |         ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:28:37: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   28 |     void define_simulation_monitor( pybind11::module &module )
      |                                     ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:30:9: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   30 |         pybind11::class_< geode::StatisticsTracker >(
      |         ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:30:52: error: [clang-diagnostic-error]

expected '(' for function-style cast or type construction
```
   30 |         pybind11::class_< geode::StatisticsTracker >(
      |                           ~~~~~~~~~~~~~~~~~~~~~~~~ ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:32:19: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   32 |             .def( pybind11::init< geode::index_t >(),
      |                   ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:32:50: error: [clang-diagnostic-error]

expected '(' for function-style cast or type construction
```
   32 |             .def( pybind11::init< geode::index_t >(),
      |                                   ~~~~~~~~~~~~~~ ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:32:52: error: [clang-diagnostic-error]

expected expression
```
   32 |             .def( pybind11::init< geode::index_t >(),
      |                                                    ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:33:17: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   33 |                 pybind11::arg( "nb_energy_terms" ),
      |                 ^
```

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:36:46: error: [clang-diagnostic-error]

use of class template 'geode::StatisticsTracker' requires template arguments

   36 |             .def( "add_realization", &geode::StatisticsTracker::add_realization,
      |                                              ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/inference/statistics_tracker.hpp:13:11: note: template is declared here
   12 |     template < typename ObjectType >
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   13 |     class StatisticsTracker
      |           ^

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:37:17: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   37 |                 pybind11::arg( "values" ),
      |                 ^
```

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:39:46: error: [clang-diagnostic-error]

use of class template 'geode::StatisticsTracker' requires template arguments

   39 |             .def( "statiscal_count", &geode::StatisticsTracker::statiscal_count,
      |                                              ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/inference/statistics_tracker.hpp:13:11: note: template is declared here
   12 |     template < typename ObjectType >
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   13 |     class StatisticsTracker
      |           ^

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:41:54: error: [clang-diagnostic-error]

use of class template 'geode::StatisticsTracker' requires template arguments

   41 |             .def_property_readonly( "means", &geode::StatisticsTracker::means,
      |                                                      ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/inference/statistics_tracker.hpp:13:11: note: template is declared here
   12 |     template < typename ObjectType >
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   13 |     class StatisticsTracker
      |           ^

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:44:25: error: [clang-diagnostic-error]

use of class template 'geode::StatisticsTracker' requires template arguments

   44 |                 &geode::StatisticsTracker::variances,
      |                         ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/inference/statistics_tracker.hpp:13:11: note: template is declared here
   12 |     template < typename ObjectType >
      |     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   13 |     class StatisticsTracker
      |           ^

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_monitor.hpp:46:41: error: [clang-diagnostic-error]

use of class template 'geode::StatisticsTracker' requires template arguments; argument deduction not allowed in lambda parameter

   46 |             .def( "__repr__", []( const geode::StatisticsTracker &self ) {
      |                                         ^~~~~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/inference/statistics_tracker.hpp:13:11: note: template is declared here
   13 |     class StatisticsTracker
      |           ^

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:28:37: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   28 |     void define_simulation_printer( pybind11::module &module )
      |                                     ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:30:9: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   30 |         pybind11::class_< SimulationPrinterConfigurator >(
      |         ^
```

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:30:27: error: [clang-diagnostic-error]

'SimulationPrinterConfigurator' does not refer to a value

   30 |         pybind11::class_< SimulationPrinterConfigurator >(
      |                           ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:37:12: note: declared here
   37 |     struct SimulationPrinterConfigurator
      |            ^

bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:32:19: error: [clang-diagnostic-error]

use of undeclared identifier 'pybind11'
```
   32 |             .def( pybind11::init<>() )
      |                   ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:32:34: error: [clang-diagnostic-error]

expected expression
```
   32 |             .def( pybind11::init<>() )
      |                                  ^
```
bindings/python/src/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:32:36: error: [clang-diagnostic-error]

expected expression
```
   32 |             .def( pybind11::init<>() )
      |                                    ^
```
include/geode/stochastic/inference/statistics_tracker.hpp:16:9: warning: [google-explicit-constructor]

single-argument constructors must be marked explicit to avoid unintentional implicit conversions
```
   16 |         StatisticsTracker( const Model< ObjectType >& model ) : model_{ model }
      |         ^
      |         explicit 
```
include/geode/stochastic/inference/statistics_tracker.hpp:32:17: warning: [readability-qualified-auto]

'auto &value' can be declared as 'const auto &value'
```
   32 |                 auto& value = values[value_id];
      |                 ^
      |                 const 
```
include/geode/stochastic/sampling/direct/object_set_sampler/point_set_sampler.hpp:39:9: warning: [google-explicit-constructor]

single-argument constructors must be marked explicit to avoid unintentional implicit conversions
```
   39 |         UniformPointSetSampler( const SpatialDomain< dimension >& domain )
      |         ^
      |         explicit 
```

include/geode/stochastic/sampling/direct/object_set_sampler/point_set_sampler.hpp:48:13: warning: [cppcoreguidelines-prefer-member-initializer]

'step_move_' should be initialized in a member initializer of the constructor

   40 |             : ObjectSetSampler< Point< dimension > >{}, domain_( domain )
      |                                                                          
      |                                                                          , step_move_(define_step_for_move())
   41 |         {
   42 |             auto volume = domain_.extended_n_volume();
   43 |             OpenGeodeStochasticStochasticException::check_exception(
   44 |                 volume != 0., nullptr, OpenGeodeException::TYPE::data,
   45 |                 "[UniformPointSetSampler] Undefined Extended Bounding "
   46 |                 "Box (volume ==0)." );
   47 |             this->log_pdf_ = -std::log( volume );
   48 |             step_move_ = define_step_for_move();
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

include/geode/stochastic/sampling/direct/object_set_sampler/point_set_sampler.hpp:69:31: warning: [readability-identifier-naming]

invalid case style for constexpr variable 'max_try'

   69 |             constexpr index_t max_try{ 100 };
      |                               ^~~~~~~
      |                               MAX_TRY
   70 |             for( const auto try_id : geode::Range{ max_try } )
      |                                                    ~~~~~~~
      |                                                    MAX_TRY

include/geode/stochastic/sampling/direct/object_set_sampler/point_set_sampler.hpp:88:28: warning: [cppcoreguidelines-avoid-magic-numbers]

0.1 is a magic number; consider replacing it with a named constant
```
   88 |             double ratio = 0.1;
      |                            ^
```

include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:40:13: warning: [bugprone-easily-swappable-parameters]

2 adjacent parameters of 'UniformSegmentSetSampler' of similar type ('const DoubleSampler::Distribution &') are easily swapped by mistake

   40 |             const DoubleSampler::Distribution& length,
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   41 |             const DoubleSampler::Distribution& azimuth )
      |             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:40:48: note: the first parameter in the range is 'length'
   40 |             const DoubleSampler::Distribution& length,
      |                                                ^~~~~~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:41:48: note: the last parameter in the range is 'azimuth'
   41 |             const DoubleSampler::Distribution& azimuth )
      |                                                ^~~~~~~

include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:65:28: warning: [cppcoreguidelines-avoid-magic-numbers]

0.1 is a magic number; consider replacing it with a named constant
```
   65 |             double ratio = 0.1;
      |                            ^
```

include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:75:31: warning: [readability-identifier-naming]

invalid case style for constexpr variable 'max_try'

   75 |             constexpr index_t max_try{ 100 };
      |                               ^~~~~~~
      |                               MAX_TRY
   76 |             for( const auto try_id : geode::Range{ max_try } )
      |                                                    ~~~~~~~
      |                                                    MAX_TRY

include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:76:29: warning: [clang-analyzer-deadcode.DeadStores]

Value stored to 'try_id' during its initialization is never read

   76 |             for( const auto try_id : geode::Range{ max_try } )
      |                             ^~~~~~ ~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:76:29: note: Value stored to 'try_id' during its initialization is never read
   76 |             for( const auto try_id : geode::Range{ max_try } )
      |                             ^~~~~~ ~

include/geode/stochastic/sampling/direct/object_set_sampler/segment_set_sampler.hpp:94:9: warning: [modernize-use-nodiscard]

function 'is_valid_object' should be marked [[nodiscard]]
```
   94 |         bool is_valid_object( const OwnerSegment2D& obj ) const override
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/helpers/fracture_simulation_runner.hpp:28:10: error: [clang-diagnostic-error]

'geode/stochastic/models/energy_terms/density_term.hpp' file not found
```
   28 | #include <geode/stochastic/models/energy_terms/density_term.hpp>
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
include/geode/stochastic/sampling/mcmc/helpers/fracture_simulation_runner.hpp:60:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
   60 |         std::string string() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/helpers/fracture_simulation_runner.hpp:87:9: warning: [cppcoreguidelines-pro-type-member-init]

constructor does not initialize these fields: set_descriptors_, set_name_to_uuid_
```
   87 |         FractureSimulationRunner( const SpatialDomain< 2 >& domain )
      |         ^
```
include/geode/stochastic/sampling/mcmc/helpers/fracture_simulation_runner.hpp:87:9: warning: [google-explicit-constructor]

single-argument constructors must be marked explicit to avoid unintentional implicit conversions
```
   87 |         FractureSimulationRunner( const SpatialDomain< 2 >& domain )
      |         ^
      |         explicit 
```
include/geode/stochastic/sampling/mcmc/helpers/fracture_simulation_runner.hpp:174:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
  174 |         std::string string() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:47:47: warning: [cppcoreguidelines-avoid-magic-numbers]

100 is a magic number; consider replacing it with a named constant
```
   47 |         index_t realisations_print_frequency{ 100 };
      |                                               ^
```
include/geode/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:51:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
   51 |         std::string string() const
      |         ^
      |         [[nodiscard]] 
```

include/geode/stochastic/sampling/mcmc/helpers/simulation_printer.hpp:83:13: warning: [modernize-pass-by-value]

pass by value and use std::move

   34 |             const SimulationPrinterConfigurator& config )
      |             ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |             SimulationPrinterConfigurator 
   35 |             : model_( model ), config_( config )
      |                                        std::move( )

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:32:17: warning: [performance-enum-size]

enum 'MHDecision' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
```
   32 |     enum struct MHDecision
      |                 ^
```

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:32:17: warning: [readability-identifier-naming]

invalid case style for enum 'MHDecision'

   32 |     enum struct MHDecision
      |                 ^~~~~~~~~~
      |                 MH_DECISION
   33 |     {
   34 |         Accepted,
   35 |         Rejected,
   36 |         Undecided
   37 |     };
   38 | 
   39 |     template < typename ObjectType >
   40 |     struct StepResult
   41 |     {
   42 |         MHDecision decision{ MHDecision::Undecided };
      |         ~~~~~~~~~~           ~~~~~~~~~~
      |         MH_DECISION          MH_DECISION

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:34:9: warning: [readability-identifier-naming]

invalid case style for enum constant 'Accepted'
```
   34 |         Accepted,
      |         ^~~~~~~~
      |         accepted
```
include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:35:9: warning: [readability-identifier-naming]

invalid case style for enum constant 'Rejected'
```
   35 |         Rejected,
      |         ^~~~~~~~
      |         rejected
```

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:36:9: warning: [readability-identifier-naming]

invalid case style for enum constant 'Undecided'

   36 |         Undecided
      |         ^~~~~~~~~
      |         undecided
   37 |     };
   38 | 
   39 |     template < typename ObjectType >
   40 |     struct StepResult
   41 |     {
   42 |         MHDecision decision{ MHDecision::Undecided };
      |                                          ~~~~~~~~~
      |                                          undecided

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:102:9: warning: [modernize-use-nodiscard]

function 'beta' should be marked [[nodiscard]]
```
  102 |         double beta() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:107:31: warning: [readability-identifier-length]

parameter name 'b' is too short, expected at least 3 characters
```
  107 |         void set_beta( double b )
      |                               ^
```

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:141:43: warning: [google-readability-braces-around-statements]

statement should be inside braces

  141 |             if( std::isnan( log_accept ) )
      |                                           ^
      |                                            {
  142 |                 return 0.0;
      |

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:143:36: warning: [google-readability-braces-around-statements]

statement should be inside braces

  143 |             if( log_accept >= 0.0 )
      |                                    ^
      |                                     {
  144 |                 return 1.0;
      |

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:147:39: warning: [google-readability-braces-around-statements]

statement should be inside braces

  147 |             if( log_accept < LOG_MIN )
      |                                       ^
      |                                        {
  148 |                 return 0.0;
      |

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:153:9: warning: [modernize-use-nodiscard]

function 'compute_log_accept' should be marked [[nodiscard]]
```
  153 |         const double compute_log_accept( const double deltaU,
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:153:9: warning: [readability-const-return-type]

return type 'const double' is 'const'-qualified at the top level, which may reduce code readability without improving const correctness
```
  153 |         const double compute_log_accept( const double deltaU,
      |         ^~~~~
```
include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:156:20: warning: [readability-math-missing-parentheses]

'*' has higher precedence than '+'; add parentheses to explicitly specify the order of operations
```
  156 |             return -beta_ * deltaU + proposal_probas.transition_probability();
      |                    ^~~~~~~~~~~~~~~
      |                    (              )
```

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:160:34: warning: [readability-function-size]

function 'accept_or_reject' exceeds recommended size/complexity thresholds

  160 |         StepResult< ObjectType > accept_or_reject(
      |                                  ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:160:34: note: 5 parameters (threshold 4)

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:165:25: warning: [cppcoreguidelines-missing-std-forward]

forwarding reference parameter 'apply_move' is never forwarded inside the function body
```
  165 |             ApplyMove&& apply_move ) const
      |                         ^
```

include/geode/stochastic/sampling/mcmc/metropolis_hasting_sampler.hpp:178:63: warning: [google-readability-braces-around-statements]

statement should be inside braces

  178 |             if( step_result.decision == MHDecision::Accepted )
      |                                                               ^
      |                                                                {
  179 |                 apply_move( state, proposal );
      |

include/geode/stochastic/sampling/mcmc/proposal/classical_proposals.hpp:34:10: warning: [readability-function-size]

function 'add_birth_death_change_moves' exceeds recommended size/complexity thresholds

   34 |     void add_birth_death_change_moves(
      |          ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/mcmc/proposal/classical_proposals.hpp:34:10: note: 6 parameters (threshold 4)

include/geode/stochastic/sampling/mcmc/proposal/classical_proposals.hpp:39:9: warning: [bugprone-easily-swappable-parameters]

2 adjacent parameters of 'add_birth_death_change_moves' of similar type ('double') are easily swapped by mistake

   39 |         double death_ratio,
      |         ^~~~~~~~~~~~~~~~~~~
   40 |         double change_ratio )
      |         ~~~~~~~~~~~~~~~~~~~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/mcmc/proposal/classical_proposals.hpp:39:16: note: the first parameter in the range is 'death_ratio'
   39 |         double death_ratio,
      |                ^~~~~~~~~~~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/mcmc/proposal/classical_proposals.hpp:40:16: note: the last parameter in the range is 'change_ratio'
   40 |         double change_ratio )
      |                ^~~~~~~~~~~~

include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:31:16: warning: [performance-enum-size]

enum 'MoveType' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
```
   31 |     enum class MoveType
      |                ^
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:31:16: warning: [readability-identifier-naming]

invalid case style for enum 'MoveType'
```
   31 |     enum class MoveType
      |                ^~~~~~~~
      |                MOVE_TYPE
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:33:9: warning: [readability-identifier-naming]

invalid case style for enum constant 'Invalid'
```
   33 |         Invalid,
      |         ^~~~~~~
      |         invalid
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:34:9: warning: [readability-identifier-naming]

invalid case style for enum constant 'Birth'
```
   34 |         Birth,
      |         ^~~~~
      |         birth
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:35:9: warning: [readability-identifier-naming]

invalid case style for enum constant 'Death'
```
   35 |         Death,
      |         ^~~~~
      |         death
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:36:9: warning: [readability-identifier-naming]

invalid case style for enum constant 'Change'
```
   36 |         Change
      |         ^~~~~~
      |         change
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:43:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
   43 |         std::string string() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:51:9: warning: [modernize-use-nodiscard]

function 'transition_probability' should be marked [[nodiscard]]
```
   51 |         double transition_probability() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:72:9: warning: [modernize-use-nodiscard]

function 'type_string' should be marked [[nodiscard]]
```
   72 |         std::string type_string() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:92:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
   92 |         std::string string() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:112:11: warning: [cppcoreguidelines-special-member-functions]

class 'Move' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator
```
  112 |     class Move
      |           ^
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:132:9: warning: [modernize-use-nodiscard]

function 'proportion_weight' should be marked [[nodiscard]]
```
  132 |         double proportion_weight() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:142:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
  142 |         virtual std::string string() const = 0;
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:160:47: warning: [cppcoreguidelines-non-private-member-variables-in-classes]

member variable 'sampler_' has protected visibility
```
  160 |         const ObjectSetSampler< ObjectType >& sampler_;
      |                                               ^
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:161:16: warning: [cppcoreguidelines-non-private-member-variables-in-classes]

member variable 'proportion_weight_' has protected visibility
```
  161 |         double proportion_weight_{ 1.0 };
      |                ^
```

include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:169:13: warning: [bugprone-easily-swappable-parameters]

2 adjacent parameters of 'BirthDeathMove' of similar type ('double') are easily swapped by mistake

  169 |             double probability,
      |             ^~~~~~~~~~~~~~~~~~~
  170 |             double birth_ratio )
      |             ~~~~~~~~~~~~~~~~~~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:169:20: note: the first parameter in the range is 'probability'
  169 |             double probability,
      |                    ^~~~~~~~~~~
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:170:20: note: the last parameter in the range is 'birth_ratio'
  170 |             double birth_ratio )
      |                    ^~~~~~~~~~~

include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:198:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
  198 |         std::string string() const override
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:246:30: warning: [cppcoreguidelines-avoid-magic-numbers]

0.5 is a magic number; consider replacing it with a named constant
```
  246 |         double birth_ratio_{ 0.5 };
      |                              ^
```
include/geode/stochastic/sampling/mcmc/proposal/moves.hpp:282:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
  282 |         std::string string() const override
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/proposal_kernel.hpp:60:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
   60 |         std::string string() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/proposal_kernel.hpp:68:11: warning: [cppcoreguidelines-special-member-functions]

class 'ProposalKernel' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator
```
   68 |     class ProposalKernel
      |           ^
```
include/geode/stochastic/sampling/mcmc/proposal/proposal_kernel.hpp:103:9: warning: [modernize-use-nodiscard]

function 'string' should be marked [[nodiscard]]
```
  103 |         std::string string() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/proposal_kernel.hpp:122:9: warning: [modernize-use-nodiscard]

function 'compute_probabilities' should be marked [[nodiscard]]
```
  122 |         std::vector< double > compute_probabilities() const
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/sampling/mcmc/proposal/proposal_kernel.hpp:144:56: warning: [readability-identifier-length]

parameter name 'p' is too short, expected at least 3 characters
```
  144 |                 probabilities.begin(), [total]( double p ) {
      |                                                        ^
```

include/geode/stochastic/sampling/mcmc/proposal/proposal_kernel.hpp:160:45: warning: [google-readability-braces-around-statements]

statement should be inside braces

  160 |             if( !cumulative_probs_.empty() )
      |                                             ^
      |                                              {
  161 |                 cumulative_probs_.back() = 1.0; // ensure exact 1.0
      |

include/geode/stochastic/spatial/object_neighborhood.hpp:30:5: error: [clang-diagnostic-error]

unknown type name 'FORWARD_DECLARATION_DIMENSION_CLASS'
```
   30 |     FORWARD_DECLARATION_DIMENSION_CLASS( BoundingBox );
      |     ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:30:42: warning: [cppcoreguidelines-avoid-non-const-global-variables]

variable 'BoundingBox' is non-const and globally accessible, consider making it const
```
   30 |     FORWARD_DECLARATION_DIMENSION_CLASS( BoundingBox );
      |                                          ^
```

include/geode/stochastic/spatial/object_neighborhood.hpp:30:42: warning: [readability-identifier-naming]

invalid case style for variable 'BoundingBox'

   30 |     FORWARD_DECLARATION_DIMENSION_CLASS( BoundingBox );
      |                                          ^~~~~~~~~~~
      |                                          bounding_box

include/geode/stochastic/spatial/object_neighborhood.hpp:31:1: warning: [llvm-namespace-comment]

namespace 'geode' not terminated with a closing comment

   31 | }
      | ^
      |   // namespace geode
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/include/geode/stochastic/spatial/object_neighborhood.hpp:28:11: note: namespace 'geode' starts here
   28 | namespace geode
      |           ^

include/geode/stochastic/spatial/object_neighborhood.hpp:35:12: warning: [cppcoreguidelines-pro-type-member-init]

constructor does not initialize these fields: index, fixed, set_id

   35 |     struct ObjectId
      |            ^
   36 |     {
   37 |         index_t index;
      |                      
      |                      {}
   38 |         bool fixed;
      |                   
      |                   {}
   39 |         uuid set_id;
      |                    
      |                    {}

include/geode/stochastic/spatial/object_neighborhood.hpp:37:9: error: [clang-diagnostic-error]

unknown type name 'index_t'
```
   37 |         index_t index;
      |         ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:39:9: error: [clang-diagnostic-error]

unknown type name 'uuid'
```
   39 |         uuid set_id;
      |         ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:57:16: error: [clang-diagnostic-error]

unknown type name 'index_t'
```
   57 |     template < index_t dimension >
      |                ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:58:11: warning: [cppcoreguidelines-special-member-functions]

class 'ObjectNeighborhood' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator
```
   58 |     class ObjectNeighborhood
      |           ^
```

include/geode/stochastic/spatial/object_neighborhood.hpp:64:25: error: [clang-diagnostic-error]

no template named 'BoundingBox'

   64 |         void add( const BoundingBox< dimension >& box, const ObjectId& id );
      |                         ^

include/geode/stochastic/spatial/object_neighborhood.hpp:64:72: warning: [readability-identifier-length]

parameter name 'id' is too short, expected at least 3 characters

   64 |         void add( const BoundingBox< dimension >& box, const ObjectId& id );
      |                                                                        ^

include/geode/stochastic/spatial/object_neighborhood.hpp:65:28: error: [clang-diagnostic-error]

no template named 'BoundingBox'
```
   65 |         void update( const BoundingBox< dimension >& old_box,
      |                            ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:66:19: error: [clang-diagnostic-error]

no template named 'BoundingBox'
```
   66 |             const BoundingBox< dimension >& new_box,
      |                   ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:67:29: warning: [readability-identifier-length]

parameter name 'id' is too short, expected at least 3 characters
```
   67 |             const ObjectId& id );
      |                             ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:68:28: error: [clang-diagnostic-error]

no template named 'BoundingBox'
```
   68 |         void update( const BoundingBox< dimension >& box,
      |                            ^
```

include/geode/stochastic/spatial/object_neighborhood.hpp:71:28: error: [clang-diagnostic-error]

no template named 'BoundingBox'

   71 |         void remove( const BoundingBox< dimension >& box, const ObjectId& id );
      |                            ^

include/geode/stochastic/spatial/object_neighborhood.hpp:71:75: warning: [readability-identifier-length]

parameter name 'id' is too short, expected at least 3 characters

   71 |         void remove( const BoundingBox< dimension >& box, const ObjectId& id );
      |                                                                           ^

include/geode/stochastic/spatial/object_neighborhood.hpp:73:9: warning: [modernize-use-nodiscard]

function 'get_all_neighbor_ids' should be marked [[nodiscard]]
```
   73 |         std::vector< ObjectId > get_all_neighbor_ids(
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/spatial/object_neighborhood.hpp:74:19: error: [clang-diagnostic-error]

no template named 'BoundingBox'
```
   74 |             const BoundingBox< dimension >& box,
      |                   ^
```
include/geode/stochastic/spatial/object_neighborhood.hpp:75:32: error: [clang-diagnostic-error]

use of undeclared identifier 'uuid'
```
   75 |             const std::vector< uuid >& targeted_set_ids,
      |                                ^
```
include/geode/stochastic/spatial/pairwise_interactions/distance_cutoff.hpp:51:9: warning: [modernize-use-nodiscard]

function 'neighborhood_searching_distance' should be marked [[nodiscard]]
```
   51 |         double neighborhood_searching_distance() const override;
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/spatial/pairwise_interactions/distance_cutoff.hpp:74:9: warning: [modernize-use-nodiscard]

function 'neighborhood_searching_distance' should be marked [[nodiscard]]
```
   74 |         double neighborhood_searching_distance() const override;
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/spatial/pairwise_interactions/pairwise_interactions.hpp:38:11: warning: [cppcoreguidelines-special-member-functions]

class 'PairwiseInteraction' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator
```
   38 |     class PairwiseInteraction
      |           ^
```
include/geode/stochastic/spatial/pairwise_interactions/pairwise_interactions.hpp:41:21: warning: [performance-enum-size]

enum 'SCOPE' uses a larger base type ('int', size: 4 bytes) than necessary for its value set, consider using 'std::uint8_t' (1 byte) as the base type to reduce its size
```
   41 |         enum struct SCOPE
      |                     ^
```
include/geode/stochastic/spatial/pairwise_interactions/pairwise_interactions.hpp:68:9: warning: [modernize-use-nodiscard]

function 'neighborhood_searching_distance' should be marked [[nodiscard]]
```
   68 |         virtual double neighborhood_searching_distance() const = 0;
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/spatial/single_object_features/segment_length_feature.hpp:35:9: warning: [modernize-use-nodiscard]

function 'evaluate' should be marked [[nodiscard]]
```
   35 |         double evaluate( const OwnerSegment2D& segment,
      |         ^
      |         [[nodiscard]] 
```
include/geode/stochastic/spatial/single_object_features/single_object_feature.hpp:31:11: warning: [cppcoreguidelines-special-member-functions]

class 'SingleObjectFeature' defines a default destructor but does not define a copy constructor, a copy assignment operator, a move constructor or a move assignment operator
```
   31 |     class SingleObjectFeature
      |           ^
```
src/geode/stochastic/sampling/direct/double_sampler.cpp:36:57: warning: [readability-identifier-length]

parameter name 'd' is too short, expected at least 3 characters
```
   36 |         std::size_t operator()( const DistributionType& d ) const noexcept
      |                                                         ^
```

src/geode/stochastic/sampling/direct/double_sampler.cpp:50:9: warning: [cert-err58-cpp]

initialization of 'distribution_registry' with static storage duration may throw an exception that cannot be caught

   50 |         distribution_registry = {
      |         ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/OpenGeode-17.0.3-rc.1-ubuntu/include/absl/container/flat_hash_map.h:190:15: note: possibly throwing constructor declared here
  190 |   using Base::Base;
      |               ^

src/geode/stochastic/sampling/direct/double_sampler.cpp:50:9: warning: [cppcoreguidelines-avoid-non-const-global-variables]

variable 'distribution_registry' is non-const and globally accessible, consider making it const
```
   50 |         distribution_registry = {
      |         ^
```
src/geode/stochastic/sampling/direct/double_sampler.cpp:50:9: warning: [misc-use-anonymous-namespace]

variable 'distribution_registry' declared 'static', move to anonymous namespace instead
src/geode/stochastic/sampling/direct/double_sampler.cpp:154:14: warning: [readability-identifier-length]

variable name 'it' is too short, expected at least 3 characters
```
  154 |         auto it = distribution_registry.find( desc.distribution_type );
      |              ^
```

src/geode/stochastic/sampling/direct/double_sampler.cpp:200:27: warning: [readability-function-cognitive-complexity]

function 'sample' has cognitive complexity of 14 (threshold 10)

  200 |     double DoubleSampler::sample(
      |                           ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:204:13: note: nesting level increased to 1
  204 |             [&engine]( auto&& d ) {
      |             ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:206:17: note: +2, including nesting penalty of 1, nesting level increased to 2
  206 |                 if constexpr( std::is_same_v< D, UniformClosed< double > > )
      |                 ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:210:17: note: +2, including nesting penalty of 1, nesting level increased to 2
  210 |                 if constexpr( std::is_same_v< D, UniformClosedOpen< double > > )
      |                 ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:214:17: note: +2, including nesting penalty of 1, nesting level increased to 2
  214 |                 if constexpr( std::is_same_v< D, Gaussian > )
      |                 ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:218:17: note: +2, including nesting penalty of 1, nesting level increased to 2
  218 |                 if constexpr( std::is_same_v< D, TruncatedGaussian > )
      |                 ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:222:17: note: +2, including nesting penalty of 1, nesting level increased to 2
  222 |                 if constexpr( std::is_same_v< D, VonMises > )
      |                 ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:226:17: note: +2, including nesting penalty of 1, nesting level increased to 2
  226 |                 if constexpr( std::is_same_v< D, TruncatedLogNormal > )
      |                 ^
/__w/OpenGeode-Stochastic/OpenGeode-Stochastic/src/geode/stochastic/sampling/direct/double_sampler.cpp:230:17: note: +2, including nesting penalty of 1, nesting level increased to 2
  230 |                 if constexpr( std::is_same_v< D, TruncatedPowerLaw > )
      |                 ^

src/geode/stochastic/sampling/direct/double_sampler.cpp:204:31: warning: [readability-identifier-length]

parameter name 'd' is too short, expected at least 3 characters
```
  204 |             [&engine]( auto&& d ) {
      |                               ^
```
src/geode/stochastic/sampling/random_engine.cpp:41:31: warning: [readability-identifier-length]

parameter name 'x' is too short, expected at least 3 characters
```
   41 |     double normal_cdf( double x )
      |                               ^
```
src/geode/stochastic/sampling/random_engine.cpp:43:16: warning: [cppcoreguidelines-avoid-magic-numbers]

0.5 is a magic number; consider replacing it with a named constant
```
   43 |         return 0.5 * std::erfc( -x / std::sqrt( 2.0 ) );
      |                ^
```
src/geode/stochastic/sampling/random_engine.cpp:43:49: warning: [cppcoreguidelines-avoid-magic-numbers]

2.0 is a magic number; consider replacing it with a named constant
```
   43 |         return 0.5 * std::erfc( -x / std::sqrt( 2.0 ) );
      |                                                 ^
```
src/geode/stochastic/sampling/random_engine.cpp:50:36: warning: [readability-identifier-length]

parameter name 'p' is too short, expected at least 3 characters
```
   50 |     double normal_quantile( double p )
      |                                    ^
```
src/geode/stochastic/sampling/random_engine.cpp:58:29: warning: [readability-identifier-length]

variable name 'a1' is too short, expected at least 3 characters
```
   58 |         static const double a1 = -3.969683028665376e+01;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:59:29: warning: [readability-identifier-length]

variable name 'a2' is too short, expected at least 3 characters
```
   59 |         static const double a2 = 2.209460984245205e+02;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:60:29: warning: [readability-identifier-length]

variable name 'a3' is too short, expected at least 3 characters
```
   60 |         static const double a3 = -2.759285104469687e+02;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:61:29: warning: [readability-identifier-length]

variable name 'a4' is too short, expected at least 3 characters
```
   61 |         static const double a4 = 1.383577518672690e+02;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:62:29: warning: [readability-identifier-length]

variable name 'a5' is too short, expected at least 3 characters
```
   62 |         static const double a5 = -3.066479806614716e+01;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:63:29: warning: [readability-identifier-length]

variable name 'a6' is too short, expected at least 3 characters
```
   63 |         static const double a6 = 2.506628277459239e+00;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:65:29: warning: [readability-identifier-length]

variable name 'b1' is too short, expected at least 3 characters
```
   65 |         static const double b1 = -5.447609879822406e+01;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:66:29: warning: [readability-identifier-length]

variable name 'b2' is too short, expected at least 3 characters
```
   66 |         static const double b2 = 1.615858368580409e+02;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:67:29: warning: [readability-identifier-length]

variable name 'b3' is too short, expected at least 3 characters
```
   67 |         static const double b3 = -1.556989798598866e+02;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:68:29: warning: [readability-identifier-length]

variable name 'b4' is too short, expected at least 3 characters
```
   68 |         static const double b4 = 6.680131188771972e+01;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:69:29: warning: [readability-identifier-length]

variable name 'b5' is too short, expected at least 3 characters
```
   69 |         static const double b5 = -1.328068155288572e+01;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:71:29: warning: [readability-identifier-length]

variable name 'c1' is too short, expected at least 3 characters
```
   71 |         static const double c1 = -7.784894002430293e-03;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:72:29: warning: [readability-identifier-length]

variable name 'c2' is too short, expected at least 3 characters
```
   72 |         static const double c2 = -3.223964580411365e-01;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:73:29: warning: [readability-identifier-length]

variable name 'c3' is too short, expected at least 3 characters
```
   73 |         static const double c3 = -2.400758277161838e+00;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:74:29: warning: [readability-identifier-length]

variable name 'c4' is too short, expected at least 3 characters
```
   74 |         static const double c4 = -2.549732539343734e+00;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:75:29: warning: [readability-identifier-length]

variable name 'c5' is too short, expected at least 3 characters
```
   75 |         static const double c5 = 4.374664141464968e+00;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:76:29: warning: [readability-identifier-length]

variable name 'c6' is too short, expected at least 3 characters
```
   76 |         static const double c6 = 2.938163982698783e+00;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:78:29: warning: [readability-identifier-length]

variable name 'd1' is too short, expected at least 3 characters
```
   78 |         static const double d1 = 7.784695709041462e-03;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:79:29: warning: [readability-identifier-length]

variable name 'd2' is too short, expected at least 3 characters
```
   79 |         static const double d2 = 3.224671290700398e-01;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:80:29: warning: [readability-identifier-length]

variable name 'd3' is too short, expected at least 3 characters
```
   80 |         static const double d3 = 2.445134137142996e+00;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:81:29: warning: [readability-identifier-length]

variable name 'd4' is too short, expected at least 3 characters
```
   81 |         static const double d4 = 3.754408661907416e+00;
      |                             ^
```
src/geode/stochastic/sampling/random_engine.cpp:86:9: warning: [readability-isolate-declaration]

multiple declarations in a single statement reduces readability
```
   86 |         double q, r;
      |         ^~~~~~~~~~~~
```

src/geode/stochastic/sampling/random_engine.cpp:86:16: warning: [cppcoreguidelines-init-variables]

variable 'q' is not initialized

   86 |         double q, r;
      |                ^
note: this fix will not be applied because it overlaps with another fix

src/geode/stochastic/sampling/random_engine.cpp:86:16: warning: [readability-identifier-length]

variable name 'q' is too short, expected at least 3 characters

src/geode/stochastic/sampling/random_engine.cpp:86:19: warning: [cppcoreguidelines-init-variables]

variable 'r' is not initialized

   86 |         double q, r;
      |                   ^
note: this fix will not be applied because it overlaps with another fix

Have any feedback or feature suggestions? Share it here.

feat(StraussTest): reactivate Strauss and Poisson Tests.

Co-authored-by: Arnaud Botella <arnaud.botella@geode-solutions.com>

feat(SimulationContext): remove inheritance in simulation runner

francoisbonneau and others added 13 commits March 27, 2026 09:04

file reoganisation

d42a8b7

feat(ObjectSets): add map registry in ObjectSets.

b3e9957

feat(Model): create a factory for energy terms.

keep refactoring

5b95c35

feat(EnergyTerm): builder for energy terms

3d7e55c

Apply prepare changes

4e4a5a6

static monitoring

14e2055

Apply prepare changes

964ff1f

feat(Helpers): create inference helpers.

a1b1ea5

Merge remote-tracking branch 'origin/HEAD' into energy_term_refactoring

b4cfae3

Apply prepare changes

2c94303

update to last OpenGeode Version

1dd751a

fix wind compile

538826d

Merge pull request #36 from Geode-solutions/energy_term_refactoring

79a054d

feat(EnergyTerm): builder for energy terms

francoisbonneau requested review from BotellaA and panquez May 25, 2026 11:40

panquez approved these changes May 25, 2026

View reviewed changes

francoisbonneau and others added 13 commits May 25, 2026 17:17

PA comments

43b70fb

Apply prepare changes

adfec09

feat(statisticstools): implement validator and quadratic loss.

8ddb2df

feat(StatsticTool): Implement validator

fda9aff

Apply prepare changes

f534c2c

fix clang error

9f30eda

try linking issue

866cb84

feat(StraussTest): reactivate Strauss and Poisson Tests.

64cc95d

Apply prepare changes

07bdc97

tidy 00

cf5d658

Apply prepare changes

8a30a3f

tidy01

ddab952

Apply prepare changes

2e072dd

francoisbonneau and others added 24 commits May 26, 2026 19:07

tidy02

c9128a9

tidy 04

405e0e4

Apply prepare changes

d436520

tidy05

0a32698

reduce nb iteration

a813370

modif param

4c2ec76

avoid generic capture in lambdas

683ebba

tidy 05

60b8dca

tidy06

40339c3

tidy07

94c997a

Merge pull request #40 from Geode-solutions/update_test_strauss

3331379

feat(StraussTest): reactivate Strauss and Poisson Tests.

feat(SimulationContext): remove inheritance in simulation runner

dbf2fdf

Apply prepare changes

1dd8903

fix execution

449ca08

tidy

3c30a67

Apply prepare changes

e0a5a94

create simulation context builder

d724578

tidy

68f5d6f

Apply prepare changes

7d1a5be

tidy2

fd01ad8

Update include/geode/stochastic/sampling/mcmc/simulation_runner.hpp

426a3a2

Co-authored-by: Arnaud Botella <arnaud.botella@geode-solutions.com>

Apply prepare changes

3f3192a

more tidy

2ae78a9

Merge pull request #41 from Geode-solutions/refacto_simulation_runner

8e86a7b

feat(SimulationContext): remove inheritance in simulation runner

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Mcmc config#39

Mcmc config#39
francoisbonneau wants to merge 50 commits into
nextfrom
mcmc_config

francoisbonneau commented May 25, 2026

Uh oh!

github-actions Bot commented May 25, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

francoisbonneau commented May 25, 2026

Uh oh!

github-actions Bot commented May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Cpp-Linter Report ⚠️

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions Bot commented May 25, 2026 •

edited

Loading