Skip to content
This repository has been archived by the owner on May 26, 2024. It is now read-only.

Clean up BOP_block_Registrator and convert it to RA2 #761

Merged

Conversation

chochem
Copy link
Member

@chochem chochem commented Oct 11, 2023

Just a small cleanup with RA2 conversion.

  • converted recipes to RA2
  • removed duplicate sapling recipes (all are added by GT too)
  • removed duplicate Pulverisation and Lathe recipes (all are added by GT too)
  • removed unused themal expansion sawmill compat
  • use proper GT method for Pyrolyse Oven recipes (instead of an outdated copied over variant with wrong circuits)
  • some minor code cleanup

@Dream-Master Dream-Master requested a review from a team October 11, 2023 19:50
Comment on lines +128 to +135
GT_Values.RA.stdBuilder().itemInputs(new ItemStack(aStack.getItem(), 1, i))
.itemOutputs(
GT_Utility.copyAmount(
GT_Mod.gregtechproxy.mNerfedWoodPlank ? ((long) tStack.stackSize)
: (((long) tStack.stackSize) * 5L / 4),
tStack),
GT_OreDictUnificator.get(OrePrefixes.dust, Materials.Wood, 2L))
.duration(10 * SECONDS).eut(8).addTo(sCutterRecipes);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this have no fluid input? Wasn't this water? Because otherwise the machines could sometimes just do this recipes instead of the one with lubricant

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just ra2ed that part :P
no fluid should generate the 3 water/distilled/lubricant recipes (its the rare case where RA2 is not explicit).
but that begs the question what the lubricant one is for. will remove that one

Comment on lines +161 to +168
GT_Values.RA.stdBuilder().itemInputs(GT_Utility.copyAmount(1L, aStack))
.itemOutputs(
GT_Utility.copyAmount(
GT_Mod.gregtechproxy.mNerfedWoodPlank ? ((long) tStack2.stackSize)
: (((long) tStack2.stackSize) * 5L / 4),
tStack2),
GT_OreDictUnificator.get(OrePrefixes.dust, Materials.Wood, 2L))
.duration(10 * SECONDS).eut(8).addTo(sCutterRecipes);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above. Shouldn't water be an input?

Copy link
Member

@BlueWeabo BlueWeabo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the cleanup

@Dream-Master Dream-Master merged commit b7e24eb into GTNewHorizons:master Oct 12, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants