Skip to content
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.

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

Increase max distance for waypoint solver. #3169

Merged
merged 1 commit into from
Sep 21, 2023

Conversation

DanAlbert
Copy link
Member

1000km isn't large enough in the case where there's an off-map spawn that's a long way from the target, but still in range for aircraft like the B-1. Double it, which for now is enough to fix the one pathological case we know.

Fixes #3156.

1000km isn't large enough in the case where there's an off-map spawn
that's a long way from the target, but still in range for aircraft like
the B-1. Double it, which for now is enough to fix the one pathological
case we know.

Fixes dcs-liberation#3156.
@DanAlbert DanAlbert enabled auto-merge (rebase) September 21, 2023 04:54
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Merging #3169 (7c4dd23) into develop (3862ec1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #3169   +/-   ##
========================================
  Coverage    45.77%   45.77%           
========================================
  Files          472      472           
  Lines        25741    25741           
  Branches      4508     4508           
========================================
  Hits         11783    11783           
  Misses       13937    13937           
  Partials        21       21           
Files Changed Coverage Δ
game/flightplan/waypointstrategy.py 96.59% <100.00%> (ø)

@DanAlbert DanAlbert merged commit e43874e into dcs-liberation:develop Sep 21, 2023
@DanAlbert DanAlbert deleted the larger-max-distance branch September 21, 2023 07:26
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.

"First input geometry is empty" error upon clicking Begin Campaign in Operation Vectron's Claw
1 participant