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

Jump to bottom

Train Depot changes (interface/wagon replace) #5187

Closed
DorpsGek opened this issue May 13, 2012 · 2 comments
Closed

Train Depot changes (interface/wagon replace) #5187

DorpsGek opened this issue May 13, 2012 · 2 comments
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)

Comments

@DorpsGek
Copy link
Member

Tve4 opened the ticket and wrote:

  1. Make the train length restriction (currently in advanced settings) accessible also/directly through the train depot interface.

  2. Also make it possible to create individual length restrictions for individual depots (due to different platform lengths on different lines, see # 3 for clarification).

  3. Add an option to purchase a full-length train of selected wagon type with single click or CTRL+click (especially useful for creating looooong unit trains in multiplayer since you only need to send one command to server instead of, say, a hundred) or if available already create a tool-tip/button for it. Automatically add the latest model of brake van if needed to the rear.

  4. When moving a large number of wagons of the same type from one row/train to another, and the train length will be exceeded because of the move, move as many wagons as possible (starting from left) instead of not moving at all. If the move includes locomotives and wagons, move the locomotives and then as many wagons as possible.

  5. When train of single type of wagons (plus brake van) goes in to depot for locomotive/wagon replacement, and it would become shorter than it was before, add the necessary number of proper wagons to retain original train length. Also remove the brake van if it won't be needed anymore and replace it with the proper wagon.

Just my two cents...

Reported version: Version?
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/5187
@DorpsGek
Copy link
Member Author

andythenorth wrote:

Very interesting suggestion, thanks.

Having a per-depot train length restriction is likely too complex. The user interface is creaking, adding more buttons to depots is not desirable for an affordance that most players don't need to use. Also in the case of depots serving multiple stations of different lengths, the affordance is of no use.

There would also be substantial design complexity here:

- Can a train of e.g. 8 tiles enter a depot with a limit of e.g. 5? Is it banned from routing to that depot, and if so, does that trigger a warning message? Or can it enter the depot, but then is stuck there until the player removes wagons?
- What happens if a train of 8 tiles has an existing goto-depot order, and the player then limits that depot to 5 tiles? Does that require a warning message? Is that an invalid order?
- Would new conditional orders be required for players who use conditional orders to control servicing?
- How does this interact with auto-replace, for the case where the trains get longer? Does it simply remove wagons as per now, to the depot limit?
- How does this interact with newgrfs which change vehicle length on a callback, using arbitrary rules which aren't predictable in advance?
- Would AIs need to be able to set and read depot length restrictions?
- Would players who buy another company request a feature to mass-remove all depot length restrictions?
- Does there need to be a global depot list, showing the length restriction for each depot?
- Multiplayer server admins will need a setting to allow or disable depot length limits.

It's 5 years since the OP added this. It's an interesting suggestion, especially the auto-fill wagons to train length, and it would be an alternative route to my pet project of improving the way train lengths are displayed.

Ultimately I think it's better met by having templated consist management, which would give the player roughly the same control requested here, alongside a lot of other useful affordances. So closing this one.

This comment was imported from FlySpray: https://bugs.openttd.org/task/5187#comment14531

@DorpsGek
Copy link
Member Author

andythenorth closed the ticket.

Reason for closing: Won't implement

See comments - nice suggestion, but too complex, and so not a current goal for OpenTTD.

This comment was imported from FlySpray: https://bugs.openttd.org/task/5187

@DorpsGek DorpsGek added component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) enhancement labels Apr 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
component: interface This is an interface issue flyspray This issue is imported from FlySpray (https://bugs.openttd.org/)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@DorpsGek