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

[WIP] New train recipes #634

Draft
wants to merge 5 commits into
base: master-ceu
Choose a base branch
from

Conversation

Jayx20
Copy link

@Jayx20 Jayx20 commented Nov 17, 2023

What

This adds new train recipes so that people can use steam trains in LV, diesel trains in MV, and electric/monorail trains in HV or above. This is also just a test PR because I haven't contributed anything before and I have 0 idea if this will be accepted.

The goal is to separate the trains out into different tiers/roles (carrying capacity vs speed, and also steam vs diesel vs electric or future technologies) and make their crafting recipes feel less random.

Also this isn't done, I'm posting this as a draft to make sure it isn't a waste of effort.

Implementation Details

Added recipes to craft train wheels, which form bogies, and also steam engines. These are used to create steam locomotives. Diesel trains would be similar but use diesel engines and the same wheels (or perhaps motorized bogies, just made by adding motors to the bogie item).

The idea is to add complexity to the crafting materials but make them reusable between different train types. Not sure how much or how little complexity is desired so I added some to the wheels/engines but left the hulls as "provide 24 steel plates".
If more complexity is desired the wheels could probably be made more interesting than just "hammer and lathe steel".

I made all the sprites, but the bogie sprite is bad and the steam engine sprite is just a placeholder.

Outcome

You can craft more types of trains

Additional Information a.k.a. recipe screenshots

engines

image

freight car

image

wheel recipes

image

@loxoDev
Copy link
Collaborator

loxoDev commented Feb 11, 2024

Last time we talked about trains, the consensus was that we would not add recipes for anything apart from the monorail... Ill ask the others if this should be merged. We can just make it a config option so the recipes can be turned on and off.

@hiimk3v
Copy link

hiimk3v commented Mar 11, 2024

I'd really like to see trains added. I'm sure a lot of other people would too.

@loxoDev
Copy link
Collaborator

loxoDev commented Mar 16, 2024

Hmm, alright. Let's just keep this PR open then.

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