Lucreeper74 / Create-Metallurgy

Add-on for Create Mod, adding new metallurgy processings.
https://modrinth.com/mod/create-metallurgy
MIT License
12 stars 8 forks source link

[Enhancement 💡] Add missing tags for more mod compatibility #10

Closed ffuentesm closed 2 months ago

ffuentesm commented 3 months ago

Describe the solution you'd like Add the forge:moltern_{material} tag to the fluids/molten buckets so the fluids are interchangeable with other mods that use molten metal fluids. (e.i. Tinkers'Construct) Also, remove the Minecraft:lava tag from the molten metal fluids, this causes the molten fluids to be eligible to be "Liquid Fuel" in mods that use Lava as a fuel source.

For the coal coke, could you use the forge:coal_coke and minecraft:coal tags?

You are also missing the forge:storage_block/{material} tag for raw Tungsten block, Tungsten block, Steel Block, and coal Coke block. And, even if you use wolframite for the item name, the tag compatibility would call the Tungsten naming for most mods.

Also, could you add a compatibility to Create:Vintage & improvements Belt Grinder?

Lucreeper74 commented 3 months ago

Hi @ffuentesm!

Also, thanks to taking the time to propose enhancements, that help me to improve Create Metallurgy 👍

ffuentesm commented 3 months ago

Thanks for the fixes.

I mainly look at Immersive engineering and Thermal Foundations for their use of the tags. if you were using minecraft:lava tag for easy fluid physics, I recall that 1.19.2 added a fluid API for that, so you could add your own physics to fluids without tagging it as lava or water.

And yes for Create: V&I, just make the recipes compatible between both belt grinders, since both mods add the same machine.

Lucreeper74 commented 3 months ago

Thanks for the fixes.

I mainly look at Immersive engineering and Thermal Foundations for their use of the tags. if you were using minecraft:lava tag for easy fluid physics, I recall that 1.19.2 added a fluid API for that, so you could add your own physics to fluids without tagging it as lava or water.

And yes for Create: V&I, just make the recipes compatible between both belt grinders, since both mods add the same machine.

I was using minecraft:lava tag since the initial commit, I wanted to see if it could help with the behavior of the liquid so that it burns the player, in the end no and I left it without really asking myself the question. I will make molten fluids burning behavior soon so thx, I will take a look to it.

I have talk with Create:V&I dev to ask him if I could add a similar block cause I planned to add specific features especially with my mod so yea, its duplicated when use both mod.

I will keep this issue open until I added the tags and maybe consider compatibility with Create:V&I.

ffuentesm commented 3 months ago
  • For molten fluids there is already tags "forge:molten_{material}" added on this commit and available in v0.0.3 of Create Metallurgy.

Are you sure? I am on version 0.0.3 on 1.19.2, and when I check the fluid tag on the molten buckets, I only get the lava tag. image

I am using Mantle to get the fluid tags from the items, and I am comparing them to the TiC Monten Buckets, since TiC adds many compatible fluids to the game.

As you can see, the Metallurgy bucket lacks the molten fluid tags. image

The same happens to other materials. image image

Also, on your basin and casting table, could you use fluid tags as input instead of item id? That way users could use fluids from other mods on the tables/basin

Lucreeper74 commented 3 months ago

Nevermind, I'm stupid. I forgot to data gen that will add tags to the game, I just added the instruction for the data generator. And for tags in input of recipes added by Create Metallurgy, this is planned. A few minutes ago, another person propose it on Discord too.

Would work properly now Commit

Lucreeper74 commented 2 months ago

Hey @ffuentesm :wave:! I have added "Forge:storage_blocks/materials" and change coke tag from "Forge:coke" to "Forge:coal_coke" in this commit