Skip to content

Conversation

@tomkcook
Copy link

Don't know if you're accepting PRs but wanted to suggest this. It has very minimal play testing so far so don't merge just yet but wanted to float the idea.

Basically, it adds a configuration option to the recipe combinator to output the ingredients of all input signals instead of just the one with the highest value. I find this useful in combination with LTN to request the train network to stock up a warehouse with the ingredients for all the locally-produced outputs.

I'm planning some play testing tomorrow. Let me know if merging this is a possibility please.

@tomkcook
Copy link
Author

tomkcook commented Jan 6, 2023

I've added another feature - the crafting combinator can now output the ingredients for the current recipe that the machine doesn't yet have.

I've now tested these a fair bit and am happy they work and don't impact other functionality.

I'm a bit less sure about the impact on UPS - the change to the crafting combinator should be low-impact but I'm a bit less certain of the impact of the recipe combinator change. I'm not very experienced in this area so input would be welcome.

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.

1 participant