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

Add: Configurable ship curve penalties (YAPF) #7289

Merged
merged 2 commits into from Mar 8, 2019

Conversation

@PeterN
Copy link
Member

PeterN commented Feb 27, 2019

Add separate penalties for 45- and 90-degree curves for ships, similar to trains. Default penalties are the same, but may want tweaking.

This makes ships prefer a route with 45-degree turns, but they can still take 90-degree turns and not get stuck.

@SamuXarick

This comment has been minimized.

Copy link
Contributor

SamuXarick commented Mar 1, 2019

Tested penalties values: 400, 500, 600
little hardinghead transport 2010-11-11

@nielsmh

This comment has been minimized.

Copy link
Contributor

nielsmh commented Mar 1, 2019

This + #7232 please :)

@PeterN

This comment has been minimized.

Copy link
Member Author

PeterN commented Mar 2, 2019

This + #7232 please :)

If only there was a way to make that happen...

@PeterN PeterN force-pushed the PeterN:ship-curve-penalty branch from e3d2625 to b2af26b Mar 6, 2019
@michicc
michicc approved these changes Mar 8, 2019
@michicc michicc merged commit 7ca1793 into OpenTTD:master Mar 8, 2019
8 checks passed
8 checks passed
OpenTTD CI Build #20190306.2 succeeded
Details
OpenTTD CI (Linux commit-checker) Linux commit-checker succeeded
Details
OpenTTD CI (Linux linux-amd64-clang-3.8) Linux linux-amd64-clang-3.8 succeeded
Details
OpenTTD CI (Linux linux-amd64-gcc-6) Linux linux-amd64-gcc-6 succeeded
Details
OpenTTD CI (Linux linux-i386-gcc-6) Linux linux-i386-gcc-6 succeeded
Details
OpenTTD CI (MacOS) MacOS succeeded
Details
OpenTTD CI (Windows Win32) Windows Win32 succeeded
Details
OpenTTD CI (Windows Win64) Windows Win64 succeeded
Details
@PeterN PeterN removed the needs review label Mar 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.