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

Resistor recipe change #1279

Merged
merged 1 commit into from
Nov 25, 2017
Merged

Resistor recipe change #1279

merged 1 commit into from
Nov 25, 2017

Conversation

JohannesGaessler
Copy link
Collaborator

Currently Resistor recipes only accept Coal Dust.
I think this is unneccarily restrictive.
This PR would allow to alternatively use Charcoal Dust or Carbon Dust.

I also removed the crafting recipe that uses 1x Wires instead of Fine Wires since the recipe is objectively worse and only serves as a noob trap.

@tomasbrod
Copy link

Why not graphite dust?

@MauveCloud
Copy link
Collaborator

Why not graphite dust?

Maybe because you can already electrolyze graphite dust for 4 carbon dust. (you can also electrolyze coal dust for 2 carbon dust, but it's somewhat less clear whether that's worth doing)

@Barhandar
Copy link

Don't make unrelated changes (wire-using recipe removal) in the same commit.

@JohannesGaessler
Copy link
Collaborator Author

@Barhandar The change is not unrelated.
Currently there are 2 crafting recipes for Resistors: [Coal Dust] x [1x Wire, Fine Wire].
Adding Charcoal Dust and Carbon Dust without further changes would have increased the number of recipes to 6 and only further increased the likelihood of confusion.

@Barhandar
Copy link

Barhandar commented Nov 13, 2017

The change is unrelated. PR deals with adding carbon and charcoal dusts to the recipe, not with your ideas about how more crafting actions is an acceptable and unquestionably superior way over using slightly more resources, and that the latter should not be an option. And said change is not announced in the commit, bordering on trying to sneak it in.

Adding Charcoal Dust and Carbon Dust without further changes would have increased the number of recipes to 6 and only further increased the likelihood of confusion.

Register wire-fine-wire-fine-wire-fine, every page will then have single dust type, preventing confusion from all but the most dim players - ones which you're implying are the majority, and which ones are actively filtered by other circuit changes anyway.

@JohannesGaessler
Copy link
Collaborator Author

JohannesGaessler commented Nov 13, 2017

trying to sneak it in.

If I was trying to sneak things in I wouldn't be making a PR in the first place.
I also wouldn't explicitly state that I did it in the OP.

the most dim players - ones which you're implying are the majority

Please stop putting words into my mouth.

Anyways, the possible confusion comes from there being a recipe with 1x Wire in the first place.
The implication of having two recipes like this is that one recipe needs you to craft more machines and in turn uses up less resources, like how you can make Plates in a machine at half the cost compared to crafting them with a Hammer.
For example, when I first played with the new Circuits I didn't know that you could make Fine Wires without a Wiremill.
This belief was only further reinforced by having two recipes that are identical except for the Copper cost.
If there had only been the recipe with Fine Wires it would have been obvious that you'd have to be able to craft them even without machines.

Both recipes become available at exactly the same time.
The fundamental metric for how expensive something is in GT is how long it takes the player to achieve it.
The amount of time it takes to Hammer a bunch of Plates into Foils is absolutely miniscule compared to the amount of time it takes to mine Copper.
Therefore the recipes with Fine Wires are objectively better.

@Blood-Asp Blood-Asp merged commit a2c7e31 into Blood-Asp:unstable Nov 25, 2017
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.

5 participants