L45[00:56:05] <Kane Hart> So I assume when beta 6 hits curse hits beta 5?

L46[01:17:00] <msticninja> I have info on an error running the github build on a dedicated forge/sponge server. IZapEffectRenderer is called on server load, which is obviously set for client side only. I have gotten it running, and connected clients running the same build, seems successful but my fix is definitely hacky. Let me know if you want info, or just tell me to GTFO and stop modifying your source. I am following your license requir

L47[01:17:00] <msticninja> and willing to contribute (though you don't need my hack I'm sure).

L48[02:18:30] <liach> i am fairly sure that effect renderer is client side insulated. if you have a fix we welcome prs

L49[02:19:49] <liach> @msticninja if you have craah report feel free to post

L52[03:00:14] <msticninja> Here's the crashlog from the server:https://pastebin.com/atkWTke2

L53[03:03:52] <msticninja> For the record, I have many mods, and a complicated server setup, so I have no idea yet if this is really a problem you should fix. The "fix" is just removing @SideOnly(Side.CLIENT) from the IZapEffectRenderer function. It's probably a bad hack, but I spent most of the day debugging this, so I'm stopping here for now.

L54[03:05:27] <msticninja> I will submit a pull if it's not a horrible hack, but I didn't want to submit a formal PR until researching the validity of the fix.

L56[05:14:31] <liach> uh no. it should be client only. looks like that initializer method

L57[05:46:00] <Chocohead> You can get away with an interface type being universal and the individual methods being side only, but that's not much help if you want the method both sides too

L58[06:23:11] <liach> Or a utility to initialize by side with a double supplier

L66[14:56:52] <msticninja> I know it should be client only, but I couldn't find what was trying to load it as a server side class, so I removed the side only requirement for now. I submitted a PR just so you could see what I changed exactly, but I highly doubt you actually want to merge the PR.

L68[16:55:42] <Kane Hart> hey so IC2 has 0 machine minetweaker support?

L69[17:13:04] <Forecaster> IC2 lets you define recipes on its own

L70[17:17:14] <Kane Hart> Yeah sadly not working for one recipe in the extractor. I copied it to the letter not sure why >.>

L71[17:22:45] <Forecaster> I can take a look tomorrow

L72[17:23:07] <Kane Hart> Thanks I will just post what I tried here as I think you and I have diff sleep times ?

L73[17:24:44] <Kane Hart> Message contained 4 or more newlines and was pastebined https://paste.pc-logix.com/ihivupaken

L76[23:55:02] <3TUSK> @Kane Hart that `#ic2air` should only work for ic2 items

L77[23:55:19] <3TUSK> I honestly don't think you can work around this

L78[23:55:38] <3TUSK> since ic2's ini files doesn't really allow you specify NBT iirc

L79[23:56:08] <coderbot> that's why I disabled ic2 machines in my pack and just used techreborn machines since they support crafttweaker

L80[23:56:15] <coderbot> maybe ic2 tweaker would work for his use case

L81[23:56:30] <coderbot> https://minecraft.curseforge.com/projects/ic2-tweaker?gameCategorySlug=mc-mods&projectID=311846

L82[23:56:36] <3TUSK> oh hey i am the author of that ic2 tweaked

L83[23:56:43] <3TUSK> oh hey i am the author of that ic2 tweaker [Edited]

L84[23:56:44] <coderbot> ah neat

L85[23:56:59] <3TUSK> i made that on someone's request

L86[23:57:07] <coderbot> it's a cool mod for pack makers who don't want to screw around with ic2's wack recipe format!