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

Pyrite Dust Unintentionally Iron Rich #931

Open
CactusAcolyte opened this issue Oct 5, 2022 · 4 comments · May be fixed by #933
Open

Pyrite Dust Unintentionally Iron Rich #931

CactusAcolyte opened this issue Oct 5, 2022 · 4 comments · May be fixed by #933

Comments

@CactusAcolyte
Copy link
Contributor

Pack version 1.2.2.1

Pyrite Dust (FeS2) and Impure Pile of Pyrite Dust smelts into one iron ingot per dust.

This seems unintentional because:

  1. Purified Pile of Pyrite Dust smelts into three nuggets.
  2. The electrolyzer recipe correctly portrays pyrite as being 1/3 of an iron dust.
  3. Pyrite ore smelts into one ingot but macerates into six crushed ores (a return of 1:6!).
  4. Other iron-based ores are balanced to be poorer, being somewhere between 1.3 to 2 iron per processed ore:
  • Magnetite (Fe3O4) macerates into four crushed ores, where each smelts into three iron nuggets.
  • Brown and Yellow Limonite (FeHO2) macerates into eight crushed ores, where each smelts into two iron nuggets.
  • Banded Iron (Fe2O3) macerates into six crushed ores, where each smelts into three iron nuggets.

Pure

image

Impure

image

Purified

image

image

@Exaxxion
Copy link
Collaborator

Exaxxion commented Oct 5, 2022

While I agree that it's a bit strange, this is just how GTCE works. Particularly for these basic metals I'm not too worried about it, as players tend to just smelt the ore directly or at most crush it before smelting in the early game, and you get infinite basic metal ingots from DML before you even have access to good ore processing.

@CactusAcolyte
Copy link
Contributor Author

Did some testing.
The unmodified GregTech .jar has it such that both dusts smelt into three nuggets.
Did some digging and found the change is in the Earlygame script.
The lines hadn't been touched since the original Nomifactory commit so I don't have a clue behind the original intentions.

@CactusAcolyte CactusAcolyte linked a pull request Oct 9, 2022 that will close this issue
@ghzdude
Copy link

ghzdude commented Oct 9, 2022

This change was likely made by Damien (the author of Omnifactory) well before Exa had worked on Omnifactory (now Nomifactory) so that it's easier to acquire iron for early game before DML.

@Exaxxion
Copy link
Collaborator

Exaxxion commented Oct 9, 2022

I believe that to be the case yes. I can see just above that it looks like an attempt was made to make Chalcopyrite dust give a full ingot, but unlike with Pyrite the original furnace recipe that outputs two nuggets was not removed so the addition silently(?) fails.

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 a pull request may close this issue.

3 participants