Skip to content

core: allow for 0ns simulation steps and synchronization points #274

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
MariusBgm wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

core: allow for 0ns simulation steps and synchronization points #274

MariusBgm wants to merge 2 commits into from

Conversation

@MariusBgm
Copy link
Collaborator

@MariusBgm MariusBgm commented Oct 31, 2025 •
edited
Loading

PoC: trigger a simulation step with 0ns duration, for certain message types (which implement the SIlKitMsgTrait::IsSynchronizationPoint)
TODO:

  • need to guard the behavior with handshake capabilities? -> don't send zero timestamps to participants which do not understand them
  • we should have an explicit synchronization message, independent of the NextSimTask. E.g., for flow control and backpressure in non-virtual-time participants TODO: look at barriers, e.g. the RequestReplyService.

VDanielEdwards
VDanielEdwards reviewed Nov 6, 2025
View reviewed changes
VDanielEdwards
VDanielEdwards reviewed Nov 6, 2025
View reviewed changes
VDanielEdwards
VDanielEdwards reviewed Nov 10, 2025
View reviewed changes
@MariusBgm
init sync points, testing using DataMessageEvents
8b1b2e3 
case insensitivity sucks
WIP NetSim Test has zero timesteps
merge fixups, build on linux
Add review feedback

Signed-off-by: Marius Börschig <Marius.Boerschig@vector.com>
@MariusBgm MariusBgm force-pushed the dev_synchronization_points branch from 0284d46 to 8b1b2e3 Compare November 10, 2025 08:45
@MariusBgm
fixup! init sync points, testing using DataMessageEvents case insensi…
d0b36e7 
…tivity sucks WIP NetSim Test has zero timesteps merge fixups, build on linux Add review feedback

Signed-off-by: Marius Börschig <Marius.Boerschig@vector.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@VDanielEdwards VDanielEdwards VDanielEdwards left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Design Proposal

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MariusBgm @VDanielEdwards