Skip to content

route_departOffset: support negative and random departure offsets - #18420

Open
NearCai wants to merge 4 commits into
eclipse-sumo:mainfrom
NearCai:pr/depart-offset
Open

NearCai wants to merge 4 commits into
eclipse-sumo:mainfrom
NearCai:pr/depart-offset

Conversation

@NearCai

@NearCai NearCai commented Oct 25, 2026

Copy link
Copy Markdown
Contributor

Fixes #10374.

As suggested in the issue, the departure offset tool now supports shifting departures
in both directions:

  • --random-depart-offset MIN,MAX shifts every departure by a random value chosen
    uniformly from [MIN,MAX[ (e.g. -1800,900); --seed (default 42) makes the
    sequence reproducible
  • negative values are accepted for --depart-offset as well
  • departures (and flow begin/end) are clamped to 0

Documentation and two tests (offset_random, offset_negative) are included.

Local verification: ran both cases and compared the output with the recorded
fixtures; identical.

Copilot AI balanced review requested due to automatic review settings October 25, 2026 08:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

behrisch and others added 3 commits October 28, 2026 21:52
The new tests have no stdout or stderr output. Use empty expected
files instead of files containing a newline.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

negative value or bracket for random-depart-offset

3 participants