Skip to content

Add: consist_current_power in varact2 for trains#419

Open
WolfieFox wants to merge 1 commit intoOpenTTD:masterfrom
WolfieFox:expose_cached_power
Open

Add: consist_current_power in varact2 for trains#419
WolfieFox wants to merge 1 commit intoOpenTTD:masterfrom
WolfieFox:expose_cached_power

Conversation

@WolfieFox
Copy link

OpenTTD has had these vars exposed since r3594 (Feb 2006) so I think we should be good to go on using them

@PeterN
Copy link
Member

PeterN commented Mar 16, 2026

formation isn't a word that's used anywhere else. The normal term that's already used is consist.

@WolfieFox WolfieFox changed the title Add: formation_current_power in varact2 for trains Add: consist_current_power in varact2 for trains Mar 16, 2026
@WolfieFox
Copy link
Author

Valid, renamed the var to match.

@2TallTyler
Copy link
Member

Needs updating to squash the fixup commit, and rebasing so the Black CI check passes.

@WolfieFox WolfieFox force-pushed the expose_cached_power branch from 7234c59 to 99c42d4 Compare March 16, 2026 23:26
@2TallTyler
Copy link
Member

Commit name doesn't match the format (but the PR title is correct). 🙂

@WolfieFox
Copy link
Author

Commit name doesn't match the format (but the PR title is correct). 🙂

Damnit, the rebase command prompted me three times and I didn't even parse it the second time. One moment...

OpenTTD has had these vars exposed since r3594 so I think we should be good to go on that.
@WolfieFox WolfieFox force-pushed the expose_cached_power branch from 99c42d4 to 1ee7451 Compare March 16, 2026 23:33
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.

3 participants