[Bug] [Overhauled] NuclearCraft breaks TFC: TNG heating recipes
FrederikRasmussen opened this issue · 6 comments
Okay, first of all, my apologies for being unable to find the Overhaul git.
Second, here's a doozy of a bug.
In Terrafirmacraft: The Next Generation, certain items can be heated to transform them. The most basic example is metal ores in a container - you heat the container to Brilliant White (the highest temperature) to convert ores into liquid metal for use in casting metal items.
However, with NuclearCraft: Overhauled installed, the items just void instead, they never become liquid.
I have attached three pictures of the process:
- Here we have a container heated to Yellow White**, very close to the melting point.
- Second is a shot after reaching Brilliant White. The large brown receptacle is the crucible, intended to catch liquids. You can see from the screenshot that it is empty.
- The final shot shows the tooltip of our vessel. It is now empty, and displays no liquids inside.
I have no idea why this happens, but I would very much appreciate a solution so I can play with both these great mods simultaneously! :)
Whoops, looks like I somehow never came back to this :P
I unfortuantely still can't think of why NC could be causing this issue. I don't use NBT to attach radiation values to items, so I don't think it's that. I think I asked about the TiC molten metals as I was wondering whether NC was messing with fluid registration, but maybe TFC doesn't even use that system.
Like turbo said this is a conflict with fluid registration, the way conflicts are handled within TFC is that when a fluid is found to have already been registered it will use that fluid over its own, adding its own "wrapper" around it. It is when the code tries to get the fluid wrapper where this code:
@Nonnull
@SuppressWarnings("ConstantConditions")
public static FluidWrapper getWrapper(@Nonnull Fluid fluid)
{
if (!WRAPPERS.containsKey(fluid))
{
// Should only ever get called for non-tfc fluids, but in which case prevents a null wrapper getting returned
WRAPPERS.put(fluid, new FluidWrapper(fluid, false));
}
return WRAPPERS.get(fluid);
}
Prevents the wrapper from working because of equality with HashBiMap<Fluid, FluidWrapper> testing against tfc copper Fluid and NC copper FluidMolten. Seems to be an issue caused by the fact that it goes through forges FluidStack as a NC fluid but with how forge handles it, it goes into the fluid registry again and grabs the tfc fluid out of the registry.
Hey there - the overhaul repo is the master branch, while the 1.12.2 branch is the pre-overhaul :)
I’ve not played with TFC before and certainly haven’t some any specific integration with it... does this happen with pre-overhaul NC, or perhaps other mods such as Tinkers’ Construct which also add molten metals?
Apologies for the slow reply.
I have now checked NuclearCraft non-overhaul against TFC to see if the same issue occurs - and it does! 😲 (No difference from the test I made previously. Test performed on fresh world)
I don't think TiC has any issues with this, but it also uses quite different 'heating' mechanics than TFC.
I think it is possibly a poor interaction between the radiation handling in NC (if there is such a thing - didn't get much of a chance to play with the Overhaul edition) which might interact poorly with the non-NBT method TFC:TNG uses to handle item temperatures and melting.