Add function call and full location in checker-side transitions

Fix CI tests

Merge branch 'master' of https://framagit.org/simgrid/simgrid

Better strategy documentation

Buffers should not be considered by CommWaitTransition::depends.

Update tesh files accordingly.

Use xbt::random instead of rand and srand

Better warning message for max-depth

Merge branch 'master' of https://framagit.org/simgrid/simgrid

Remove unused parameters (sonar).

Connect the right signal.

Declare functions "const" (sonar).

Another round of pointer-to-const for sonar.

Use xbt_die instead of xbt_assert(false).

Disable maybe throwing move constructor (sonar).

Merge branch 'master' of https://framagit.org/simgrid/simgrid

Sort MANIFEST.in.

[ci-skip]

Few more CI tesh fixing

Fix stat order for CI tests

Fix build without MC.

Merge branch 'master' of https://framagit.org/simgrid/simgrid

Fix Manifest.in

Clang-tidy: readability-qualified-auto.

Also add const qualifiers to pointers and references when possible.

Define classes with "class".

Postpone the destruction of Task::current_activity_.

Fixes heap-use-after-free spotted by ASan and valgrind.

Various cleanups in Task plugin.

* reduce number of calls to simcall_answered
* use xbt::signal for the "on_this_*" signals
* reduce scope for static member "inited"
* remove intrusive_ptr_* function for derived classes

Add factorisation for strategy and Use dynamic over static cast

Merge branch 'master' of https://framagit.org/mwapl/simgrid

Install package non-interactively on CI

Another struct/class mismatch between forward declaration and actual use

Enable the ODPOR/SDPOR tests in default builds also

Please clang by not defaulting a constructor that it will delete

odpor_tests_private.hpp:24:3: error: explicitly defaulted default constructor is implicitly deleted [-Werror,-Wdefaulted-function-deleted]
  DependentIfSameValueAction() = default;
    ^
odpor_tests_private.hpp:21:13: note: default constructor of 'DependentIfSameValueAction' is implicitly deleted because field 'value' of const-qualified type 'const int' would not be initialized
  const int value;
            ^

Merge branch 'master' of https://framagit.org/simgrid/simgrid

Fix stat missorder at the end of DFSexplorer

Cosmetics: whitespace cleanup (codefactor.io).

Call CommImpl::set_iface(nullptr) as soon as the comm is finished, and make TSan happy with s4u-app-masterworkers-parallel tests.

Merge branch 'fix-sonar-task' into 'master'

Fix sonar task

See merge request simgrid/simgrid!160

more sonar fix

fix sonar warnings

Merge branch 'odpor-clean-ups' into 'master'

Address Comments/Errors from ODPOR/SDPOR Integration

See merge request simgrid/simgrid!159

Fix RuntimeError: pybind11::handle::inc_ref() PyGILState_Check() failure.

See commit 2c2d94471739d82e9cbd370711f73377e0b17717 for details.

CI pages: try to use the Debian version of the python packages

Remove a test that was indended to show that this branch is never used

Complete missing docs for WakeupTreeNode

Add more documentation and fix forward declaration

Merge branch 'master' of https://framagit.org/simgrid/simgrid

Fix few typos and add random seed intializer

Use Debian:12 for our build deps

Merge branch 'rename-operation-task' into 'master'

rename operation to task

See merge request simgrid/simgrid!158

Merge branch 'odpor-implementation' into 'master'

Introduce SDPOR and ODPOR into Simgrid

See merge request simgrid/simgrid!154

fire on_start for one-sided communication instead of on_send/recv

add Comm::on_this_send and Comm::on_this_recv

[no-ci] indicate which methods and signals are static in the doc

[no-ci] precise which methods of the Actor class are static

This should be fixed now.

A deleted Comm cannot be a valid iface anymore.

rename operation to task. rename execute to fire.

properly remove the old signals and prevent MC to fire Comm::on_this_completion

react to the proper signals

Introduce ODPOR integration with multiple actions

Prior commits had missing integration with transitions
which could be executed multiple times in the form of
variants. This commit resolves those issues by making
two important modifications:

1. Determining the "current thing that actor X is
running" from a given state `s` has been modified
to reflect what we would expect if we asked the
question even after having exhausted all combinations;
the the case where all combinations have been executed,
the last variant is that which we would expect to be
considered the transition which were enabled at `s`.
This allows us to "fix" the spot in ODPOR where we
need to determine those actions enabled at a given
state `s`. We need to treat it as if only one variant
actually exists; for if there are differences in the
dependency structure for each variant, we wouldn't
want to erroneously deduce that an actor `X` were
a weak initial for some prefix `E'` of `E` and
for some extension `v` of `E'` based on what it
HAD run in the past.

2. In ODPOR, we now only insert members into the
sleep set when all variants of a transition have
been executed. This ensures that we are effectively
treating each variant in isolation from one another;
it is only when we reach the last variant that we
can finally go ahead and say "OK actor X you can
go into the sleep set now". Note that it's perfectly
possible for the actor to later end up in sleep sets
of states that occur *later on*; but for the sleep
set that is at *this* state, we'd want to be sure
only to add it after having explored the state space
with each variant. The reasoning is the same as in
point 1: we wouldn't want the sleep set to be based
on an action that the actor HAD run in the past as
the proxy representing what that actor is doing IN
THE PRESENT. If the dependencies were different among
the different variants, we'd have an issue if we
had previously added the other variants

Merge branch 'python-repr' into 'master'

add repr for many simgrid objects

See merge request simgrid/simgrid!156

switch repr to Class(objectname). fix python operation tests.

Add small test for insertions

add repr for many simgrid objects. replace mailbox __str__ to match __repr__ of others objects. clang-format.

Add odpor_tests_private.hpp to MANIFEST.in

ASan: yet another odr-violation to ignore.

Add missing includes.

Correct signature for deleted constructor.

firing on_this_completion breaks a lot of tests and I don't see how to fix this (@agiersch, any idea?)

 add on_this_veto signal to Activities

forgot to fire signals

simplify example

 add on_this_[suspend,resume] signals to Activities

 add on_this_completion signal to Activities

 add on_this_start signal to Activities

improve comment

Merge branch 'doc-photovoltaic-plugin' into 'master'

add links to photovoltaic power/cost models in docs

See merge request simgrid/simgrid!155

add links to power/cost models in docs

Add SDPOR backtracking simulation unit test

This commit adds a small unit test which
simulates the process of determining which
threads need to be added into the backtrack
set of a given state using SDPOR

Add more independence tests w.r.t a partial execution

Merge branch 'operation-python' into 'master'

add python bindings for operations

See merge request simgrid/simgrid!153

Merge branch 'simgrid-fork-plugin-photovoltaic'

Fix MANIFEST.in etc.

move on_start and on_completion from CommImpl to Comm

Force the use of sleep sets with ODPOR

GitHub action: update to actions/checkout@v3

Adjust tesh tests according to changes in deps.

A number of tests were affected by the previous
additions that labeled transitions executed by
the same actor as dependent. The appropriate
modifications were made to the expected outputs
and a "sanity" check was performed before making
them

Mark transitions run by the same actor as dependent

This commit adds a missing component to a number of
transition classes. Transitions that are executed by
the same actor are inherently dependent; but only the
RandomTransition actually checked for this case! This
caused ODPOR to search traces incorrectly under the
assumption that transitions which were actually
dependent were independent

Fix subtle bug in ~_E computation

The computation for the action ~_E requires
that we compute whether `p` is independent with
some sequence of transitions `w` after an
execution `E`. The algorithm for computing
whether an equivalent trace was found works
to iteratively eliminate events from the
equivalent sequence, leaving behind only
the remaining "bits" that are needed for
insertion into a wakeup tree.

The bug involved a typo of `w` for `w_now`
:( :( which broke everything :/

add python bindings for operations

Add remaining handlers to ReversibleRaceCalculator

This commit adds handlers for each type of transition
to ReversibleRaceCalculator which, as the name suggests,
determines whether a reversible race exists between two
events e1 and e2 based on the type of transition found
at e2. The full semantics have not been added for each
type of transition and will need to be added in the future

Resolve misconception with ODPOR pseudocode impl.

Like the SDPOR implementation before it, the ODPOR
implementation suffered from a similar fault in its
implementation. Determining whether an insertion was
necessary into a wakeup tree for some point in the
past requires determining whether the sleep set and
the set of weak initials overlap one another. If
their intersection is the empty set, THEN we attempt
to insert an actor. The prior implementation attempted
to find the first such actor that was not contained
in the sleep set, but this is semantically different

Resolve misconception with SDPOR pseudocode impl.

This commits fixes a semantics mismatch between
the implementation and the pseudocode in SDPOR.
The trouble was that we didn't stop searching
for an initial when we noticed that ANY initial
was contained in the backtrack set; instead, we
simply searched until we found ANY initial contained
OUTSIDE OF the backtrack set. The latter is not
equivalent to the former, so SDPOR was doing
more work than necessary.

A similar implementation issue is also present in
ODPOR which will be resolved in due time.

Fix subtle bug in Execution regeneration in DFSExplorer

Prior to this commit, during state regeneration,
we had used the state's `get_transition_out()`
method to refill the `Execution` instance bound
to DFSExplorer (`execution_seq_`). The trouble
is that this is an **unstable** source of truth:
it should not be trusted to regenerate an execution.
Instead, the `get_transition_in()` method should
be used on the state which follows. Intuitively,
both methods should be equivalent, which is why
the bug was overlooked for so long. Now that the
execution is correct and that SDPOR/ODPOR are
based on the correct sources of truth, things
are moving along more nicely and the two algorithms
appear to behave more sanely.

In addition to the above bug fix, this commit adds
a return value to the `insert()` function of the
`WakeupTree` class. This extra information is used
in debug logs to get a better idea of what ODPOR
is doing during its insertion. Similar debug information
will be added to SDPOR in subsequent phases.

Keep pointers to transitions instead of slices

Prior to this commit, sleep sets contained slices

Fix two minor bugs in the ODPOR implementation

This commit fixes the following two bugs:

1. When seeding a wakeup tree for the first
time, we forgot to add a `break` statement
to ensure that only one such enabled thread
were placed into the wakeup tree. Before
all such enabled threads were inserted which
amounted to what would have been a brute-force
search :(.

2. When determining whether a sequence is a
candidate for insertion into a wakeup tree,
we may need to check, for all enabled actors
in a given state, whether those actors'
next steps are independent with the sequence
theretofore constructed. This additional
step peforms the work needed to check whether
those actors are contained in the set of
*weak* initials. The method corresponds
to line 6 of the pseudocode of the ODPOR algorithm

Please Sonar and scan-build.

Make ASan ignore more odr-violations.

Fix some sonar issues in includes, after upgrade to c++17.

Disable implicit conversions in nlohmann/json.

See https://github.com/nlohmann/json#implicit-conversions

Add reversible race implementations for Comm actions