Skip to content

Boat Speed Boundary escape fix - #206

Draft
kabarmaz wants to merge 1 commit into
mainfrom
boat-speed-boundary
Draft

kabarmaz wants to merge 1 commit into
mainfrom
boat-speed-boundary

Conversation

@kabarmaz

@kabarmaz kabarmaz commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

Related Issue / Discussion:

Algorithm accepts speed values out of the range of 'BOAT_SPEED_BOUNDARIES'.

Changes:

  • Added statements to keep boat speed within boundaries while calculating speed with arrival time to population.py(line 116-121).

  • Added a test that sets the boundaries to 6-7 and verifies the recalculated route speed stays within that range and gets clamped to upper bound when needed to test_genetic.py(line 331-333)

PR Checklist:

In the context of this PR, I:

Please consider that PRs which do not meet the requirements specified in the checklist will not be evaluated. Also, PRs with no activities will be closed after a reasonable amount of time.

This branch has not been deployed

No deployments
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.

1 participant