Consistency+

Consistency+

2M Downloads

mod changes stone cutter recipes instead of adding them

TobiPeterG opened this issue · 25 comments

commented

Describe the bug
Hey there,

I noticed that consistency plus doesn't add stonecutter recipes, instead it changes the existing ones. Because of this, recipes added by other mods like adorn are shown as preview of the block, but a consistency plus block is crafted and visible in the output window. This happened after the update to version 0.4.0

Versions
Minecraft version: 1.17.1
C+ version:
Fabric API version: 0.40.0

Can this issue be confirmed?

commented

This happened after the update to version 0.4.0

We're aware this is an issue in 0.4.1-rc.1, are you sure it's present in 0.4.0 as well? I don't believe it is, because 0.4.1-rc.1 completely changes stonecutter behaviour

Uh, I'm not 100% sure.
I had 0.4.0 installed for just a short time, before I switched to 0.4.1-rc1 and meant it was on 0.4.0 as well, but then I'm probably wrong here

commented

This happened after the update to version 0.4.0

We're aware this is an issue in 0.4.1-rc.1, are you sure it's present in 0.4.0 as well? I don't believe it is, because 0.4.1-rc.1 completely changes stonecutter behaviour

commented

We’re currently working on this issue, but are unsure if it is a bug with C+ or Fabric API and what causes it. We’ve determined that the issue is present in 0.4.1rc1, and in multiplayer only, and seems to be tags not syncing between server and client, but that’s about all we know so far. We’ve also figured out that if you open a single player world, then join multiplayer, having synced the tags in a singleplayer world seems to temporarily fix it in multiplayer as well.

commented

We’re currently working on this issue, but are unsure if it is a bug with C+ or Fabric API and what causes it. We’ve determined that the issue is present in 0.4.1rc1, and in multiplayer only, and seems to be tags not syncing between server and client, but that’s about all we know so far. We’ve also figured out that if you open a single player world, then join multiplayer, having synced the tags in a singleplayer world seems to temporarily fix it in multiplayer as well.

Humm OK, that's weird, but I've exactly hit the sweet spot of the bug here, as I'm using a private server (since I can't play in single-player worlds anymore for some reasons I can't find out)

commented

can you provide screenshots of the issue?

commented

oh god it is the same issue
we've been struggling with this for a bit now. we can't find a way to reproduce it. I'm convinced there's ghosts in the code.

commented

re-open this if it comes back

commented

this is the first occurrence of it happening in singleplayer.

commented

can you provide screenshots of the issue?

So you want a screenshot of the stone cutter then I suspect? :D

commented

I'm talking about the stonecutter

commented

I'm talking about the stonecutter

Oh okay, sorry, no, that's on the server, sorry. 😅

Deleted missleading comments

commented

0.39? of what?

0.33 ofc, of c+, sorry

commented

yes, to confirm that it is the same issue we've been having

commented

grafik

commented

yep, that's it. thanks. We're trying to find a fix but we have NO idea what causes it.

commented

yep, that's it. thanks. We're trying to find a fix but we have NO idea what causes it.

OK, thanks!
Well, in 0.33 it doesn't happen, so whatever you changed here, revert it. :P

commented

0.39? of what?

commented

revert it

Well the problem is that the stonecutter Changes we made in 0.4.1rc1 were a fix for another issue that was present in 0.4/0.3.3, so reverting it would bring back another, arguably worse bug

commented

revert it

Well the problem is that the stonecutter Changes we made in 0.4.1rc1 were a fix for another issue that was present in 0.4/0.3.3, so reverting it would bring back another, arguably worse bug

Uh, I'm sorry, that was more meant as a joke, not serious 😅

commented

The reason our stonecutter recipes cause issues is because one of our developers created a custom library that can create stonecutter recipes out of item tags, rather than having an individual json for each recipe. This was a fix to the low performance we had due to the many thousand jsons just from stonecutter recipes.

OK, that's weird that this issue exists then. 🤔

commented

All other mods just use regular stonecutter recipes. C+ has a huge amount of blocks, so adding recipes for everything resulted in over 7k recipe files. Loading these on server start took over a full minute of loading. I created a library to deal with this. This library lets us define recipes with tags instead, replacing 7k files with less than 100. This bug is caused by that library, and I still have no idea how to fix it.

commented

I've created #61 to keep track of this issue from now on

commented

btw. other mods add stonecutter recipes as well, like Adorn and it's not happening in that mod.
Maybe their code might be of interest?

commented

It's less a matter of adding them. And more a matter of their performance effects

commented

The reason our stonecutter recipes cause issues is because one of our developers created a custom library that can create stonecutter recipes out of item tags, rather than having an individual json for each recipe. This was a fix to the low performance we had due to the many thousand jsons just from stonecutter recipes.