-
-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Causes reload during first data load (and incompatible with Anvil Crushing Recipes) #41
Comments
Thanks for the hint, the |
Actually after having a closer look now, I don't think that method works. It requires the datapack to be in the mod jar file, but the whole reason why I manually copy the files into the datapack directory is that I need the pack to be dynamic, depending on what rules are set the contents of that pack vary. As previously discussed in #17 (comment), I am also pretty much unable to create the datapack files before the default first load starts, because carpet only parses the rule config after that. If you have another suggestion for a potential solution or I simply misunderstood the |
My recommendation would be: for each carpet rule that controls a datapack, remove it, and replace it with a builtin datapack. The only difference for users would be that they enable/disable them with |
That won't work with rules like So, as the hard conflict between our mods seems to have been resolved on your side, I will close this now. |
This line: https://github.com/RubixDev/Rug/blob/1.19/src/main/java/de/rubixdev/rug/RugServer.java#L243
is starting a reload before the first load has finished.
This makes Rug incompatible with my mod, Anvil Crushing Recipes (issue on my side here), because of this line in ACR, which requires the assumption that one reload finishes before another begins.
I can remove the exception and instead just log an error, which I plan to do if only to prevent hard conflicts like this.
However causing an additional load really isn't a good idea. It's inefficient and can lead to strange interactions like this one with ACR.
It looks like the reload simply enables a builtin datapack. If this is the case, you could use Fabric API's
registerResourcePacks
, which I actually do in ACR here.The text was updated successfully, but these errors were encountered: