Skip to content

Polish smeltery#863

Open
Seggan wants to merge 34 commits intomasterfrom
chore/smeltery
Open

Polish smeltery#863
Seggan wants to merge 34 commits intomasterfrom
chore/smeltery

Conversation

@Seggan
Copy link
Copy Markdown
Member

@Seggan Seggan commented Mar 8, 2026

  • Updated smeltery recipes
  • Removed Smeltery Caster and made it a separate block called the Casting Unit that requires the fluid piped in
  • Smeltery Input and Output Hatches now require the correct temperature
  • Added casts, for casting liquid metals to nuggets, ingots, blocks, sheets, and pipes (pipes use the crafting ratio, so pipe bender is still more efficient)

Copy link
Copy Markdown
Member

@Intybyte Intybyte left a comment

Choose a reason for hiding this comment

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

Reviewed, works ingame, also are we sure we should drop the documenation of said recipes? I mean they are easy to understand even without it

Comment thread src/main/java/io/github/pylonmc/pylon/content/machines/smelting/CastingUnit.java Outdated
Comment thread src/main/java/io/github/pylonmc/pylon/recipes/CastingRecipe.java Outdated
Comment thread src/main/java/io/github/pylonmc/pylon/recipes/CastingRecipe.java Outdated
Comment thread build.gradle.kts
@AlmostPanda
Copy link
Copy Markdown

Just want to ask, if controller/burner (also diesel burner) switches to lit=true state when active, because currently (stable version, not PR) it does not

@Seggan
Copy link
Copy Markdown
Member Author

Seggan commented Mar 16, 2026

Reviewed, works ingame, also are we sure we should drop the documenation of said recipes? I mean they are easy to understand even without it

The docs were wrong even at that point

@Seggan
Copy link
Copy Markdown
Member Author

Seggan commented Mar 16, 2026

Just want to ask, if controller/burner (also diesel burner) switches to lit=true state when active, because currently (stable version, not PR) it does not

Oh yeah I can fix that

@Intybyte Intybyte linked an issue Mar 17, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

@LordIdra LordIdra left a comment

Choose a reason for hiding this comment

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

The molding recipes won't work because a clay ball cannot be used with the manual brick mold. May I suggest that instead of the molding recipes, we just do it how tinker's and similar mods do it, and require you to pour liquid gold or similar over a block, ingot, pipe, etc?

Apologies this took me so long to get to

Comment thread src/main/java/io/github/pylonmc/pylon/recipes/CastingRecipe.java Outdated
Comment thread src/main/resources/recipes/pylon/casting.yml
@Seggan
Copy link
Copy Markdown
Member Author

Seggan commented Mar 18, 2026

The molding recipes won't work because a clay ball cannot be used with the manual brick mold.

That's intentional, you need a diesel one for it

@LordIdra
Copy link
Copy Markdown
Contributor

The molding recipes won't work because a clay ball cannot be used with the manual brick mold.

That's intentional, you need a diesel one for it

gotcha - still, current recipes feel rather strange to me

@JustAHuman-xD
Copy link
Copy Markdown
Contributor

Fixes #580

@JustAHuman-xD JustAHuman-xD linked an issue Mar 25, 2026 that may be closed by this pull request
@Seggan Seggan requested a review from LordIdra April 6, 2026 20:15
@Seggan Seggan requested a review from Intybyte April 6, 2026 20:15
Copy link
Copy Markdown
Contributor

@LordIdra LordIdra left a comment

Choose a reason for hiding this comment

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

  • Casts and casting unit are missing researches
  • Pipe cast is actually less efficient than crafting manually - I think we should make it slightly better that crafting manually, maybe like on part with pipe bender since the smeltery is more annoying that pipe benders to automate

Does anyone have other opinions on the cast recipes? The 'craft 9 nugget molds into one nugget mold' thing does not sit right with me at all logically or gameplay-wise - still think it would be better to do something more tinkers-style with casting gold over an ingot or something. is it just me?

Otherwise looks good (pending the switch to using tags for melting points). Have not looked at code since last review but smeltery code is an indecipherable beast to me anyway XD so not going to worry about it as long as everything is good gameplay-wise

@Seggan
Copy link
Copy Markdown
Member Author

Seggan commented Apr 11, 2026

The pipe efficiency is exactly the same as crafting them, 6 sheets / 8 outputs = 0.75 ingots per pipe * 144 mb / ingot = 108 mb

I deliberately used the worse ratio of the two I order to not discourage using the pipe bender, as casting pipes is ridiculously easy once you have the tech, so I still wanted there to be a reason to use the pipe bender.

Can you clarify on what you mean "pouring over a gold ingot"? Actually I should probably go look up tinkers lol

@Seggan
Copy link
Copy Markdown
Member Author

Seggan commented Apr 11, 2026

To clarify by what I mean "easy": it's literally ingots and diesel in, pipes out, much faster than the pipe bender too

@LordIdra
Copy link
Copy Markdown
Contributor

LordIdra commented Apr 12, 2026

Here's how it works in tinker's: https://www.youtube.com/watch?v=XjFSR6bF82c

For the pipe casting recipe - mb think I misread the amount. I definitely don't think it's that easy though, like if you're smelting multiple metals with the smeltery you have to keep the right one on the bottom (I saw many people build molten metal filtering setups on the playtest for this), you have to actually fuel the smeltery as you may not have it on constantly (and if you do, that's quite expensive), you need to put the right mold in the caster and then schedule casts. Compared to putting a block of steel into a pipe bender it's quite annoying. I think if it was the same cost as pipe bender it would be much more balanced

Seggan added 6 commits April 13, 2026 12:13
# Conflicts:
#	src/main/java/io/github/pylonmc/pylon/PylonRecipes.java
#	src/main/resources/lang/en.yml
#	src/main/resources/recipes/minecraft/crafting_shaped.yml
@Seggan Seggan requested a review from LordIdra April 13, 2026 19:40
@LordIdra
Copy link
Copy Markdown
Contributor

A few remaining issues I found but all small

image Oh... that's why I thought it was 144mb not 108 lol. This will need updating. (Or removing, given that the amount is defined in the recipe not the mold?)

Pipe and block mold can be placed

Still missing researches for casts (they have the wrong key in researches.yml)

this:
image

@LordIdra
Copy link
Copy Markdown
Contributor

@Seggan just pinging to make sure you saw the above since I realised I didn't submit it as a review ^

@Seggan
Copy link
Copy Markdown
Member Author

Seggan commented Apr 17, 2026

Yewh I saw, was just busy with electricity/irl

@Seggan
Copy link
Copy Markdown
Member Author

Seggan commented Apr 20, 2026

@LordIdra done

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.

Smeltery Damage Interpolate smeltery display color and position

5 participants