BluSunrize / ImmersiveEngineering

Wires, transformers, high voltage! Bzzzzt!
Other
791 stars 395 forks source link

StackOverflow error when generating power through LV Connectors and copper wire #187

Closed darthvader45 closed 8 years ago

darthvader45 commented 9 years ago

I set up my copper wires to connect through a few LV connectors and to my LV capacitor. However, as soon as I had it set up correctly, I immediately got a StackOverflow error and could not continue.

[14:35:37] [Server thread/ERROR]: Encountered an unexpected exception net.minecraft.util.ReportedException: Ticking block entity at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:642) ~[MinecraftServer.class:?] at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:547) ~[MinecraftServer.class:?] at net.minecraft.server.integrated.IntegratedServer.func_71217_p(IntegratedServer.java:186) ~[bsx.class:?] at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:427) [MinecraftServer.class:?] at net.minecraft.server.MinecraftServer$2.run(MinecraftServer.java:685) [?:?] Caused by: java.lang.StackOverflowError at com.google.common.collect.AbstractMapBasedMultimap.containsKey(AbstractMapBasedMultimap.java:185) ~[guava-17.0.jar:?] at com.google.common.collect.ArrayListMultimap.containsKey(ArrayListMultimap.java:66) ~[guava-17.0.jar:?] at blusunrize.immersiveengineering.api.ImmersiveNetHandler.getIndirectEnergyConnections(ImmersiveNetHandler.java:219) ~[ImmersiveNetHandler.class:?] at blusunrize.immersiveengineering.common.blocks.metal.TileEntityConnectorLV.transferEnergy(TileEntityConnectorLV.java:185) ~[TileEntityConnectorLV.class:?]
hron84 commented 9 years ago

Please provide IE version, Forge version, and also bit more details and/or a picture about your setup.

BluSunrize commented 9 years ago

And you are managing my bug reports because....?

T1mothyyang commented 9 years ago

Yes we will take over the bug reports section >:D

darthvader45 commented 9 years ago

I have the current 1.7.10 version, 0.4.1, the current recommended version of Forge (10.13.4.1490), and my wires were hooked up to a windmill and a capacitor.

theunkn0wn1 commented 9 years ago

unable to reproduce against commit 5e2e3520e1d40063345f440584e57a8ce4269190, attemtped to recreate your sutation with a windmill, 5 LV connectors, and 3 LV capacitors; my setup is shown below. image

Running FTB Horizons (recommended build) with no other 3rd party mods.

EDIT: Rolled back to IE version 0.4.1, still unable to reproduce with above setup.

darthvader45 commented 9 years ago

Thank you, will download newer version and see what I come up with.

On Sat, Jul 18, 2015 at 4:48 PM, theunkn0wn1 notifications@github.com wrote:

unable to reproduce against commit 5e2e352 https://github.com/BluSunrize/ImmersiveEngineering/commit/5e2e3520e1d40063345f440584e57a8ce4269190, attemtped to recreate your sutation with a windmill, 5 LV connectors, and 3 LV capacitors; my setup is shown below. [image: image] https://cloud.githubusercontent.com/assets/3110986/8763495/7883d210-2d53-11e5-8234-2877046aca40.png

Running FTB Horizons (recommended build) with no other 3rd party mods.

— Reply to this email directly or view it on GitHub https://github.com/BluSunrize/ImmersiveEngineering/issues/187#issuecomment-122596421 .

Golden rule in Minecraft: Never dig straight down.

hron84 commented 9 years ago

@theunkn0wn1 it is not constantly reproducible and absolute not related to windmills. I created a very basic setup with a thermoelectric stuff and some TE pulverizer/furnace, and eventually wires vanished. In my case only the "power extractor" wires vanished (that are directly connected to a power generator device, including TE leadstone cell), the interconnecting wires (between pulverizer and furnace) were intact. But i'm not sure it is only my luck or the issue appears around power generator devices.

theunkn0wn1 commented 9 years ago

I used windmills in my test as that was the clearest bit of information i could derive from the OP's messages. I expect I missed something as there has to be a reason for the crash to begin with.

in my test world, i use a combination of Solar farms, watermills, and Diesel generators to power everything. So far i cannot reproduce the @darthvader45 stack overflow crash based on the information he has provided.

My setup mainly has MFR machines receiving power with EIO capacitors to buffer the power. Most of my setup is HV, stepping it down to deliver it to machines as necessary. Making some modifications to my test worlds power distribution to be more dependent on IE as i hope to repo this bug

@darthvader45 can you show us exactly what setup induced the issue?

hron84 commented 9 years ago

@theunkn0wn1 is there any chance to wire disappearing caused by too strong power?

theunkn0wn1 commented 9 years ago

Too much power frying your cables? unlikely. Power is limited per connection and wire type from my understanding. If you attempt to funnel too much power, such as an HV current into an MV line, the power would transfer at the MV rate. if wires fried, IE would be doing what IC2/E does with current, IE does not need that nor does blue appear to want to copy IC2/E image In the above setup, even though the power source and target was HV, the power was recieved at the MV rate.

one funny thing i have noted is with HV power, but may apply to M/LV: If the entire network is HV, then power transfer is limited by the connections, not the cable. for example, HV by default conducts aprox. 4KiRF/T, if i wanted to transfer more than 4KiRF/T then all i would need to do is add more HV connectors to the input/output of the system. As a result, huge amounts of power can be transferred over HV (and possibly L/MV) image this setup can produce ~5KiRF/T with a peak transfer rate of ~10KiRF/T I discovered the cable itself can transfer as much as is needed, the connectors are the bottleneck in terms of transfer rate. here is a build demonstrating this. image image Though if the client crashes in certain manners then yes items can be destroyed.

magik6k commented 9 years ago

It is most likely already fixed by #207, as solves most of issues with power.

theunkn0wn1 commented 9 years ago

was #207 merged into IE as of current?

magik6k commented 9 years ago

Nope it wasn't for some reason. Just tested and it still doesn't fix HV relay power rate, so this /may/ be intentional

theunkn0wn1 commented 9 years ago

I don't think the HV relay power rate is not intentional, even TE regulates power per connection rather than per line. TE has had tiered power transfer far longer than IE, to be fair TE existed longer than IE.... As much as the transfer rate fora single line should be capped, It should not be bound to the connection transfer rate IMHO, it should be several times that of a single connection. otherwise the current diesel generator has two redundant power outlets among other practicality issues that would arise.

hron84 commented 9 years ago

@theunkn0wn1 cool, thank you for clarifying this. I was not sure about the wires, since IE is a relative new mod and I do not have enough experience with it yet. I have no idea then why wires disappearing.

(btw, that solar farm is... pretty huge. Solar Expansion or Solar Flux)?

theunkn0wn1 commented 9 years ago

@hron84 Wires disappearing is not the OP's issue from my understanding. please create a new issue thread stating the issue, IE version, Forge version (or modpack version), and any other relevant information so we can track down your issue rather than hijacking this thread any more.

(btw, the farm is Solar Expansion)

purplemonday commented 8 years ago

issue still there current latest version on curse just created this acidental loop between termal expansion tesseracts, the network whas double MV based and included two mv capacitors i have updated all involved mods and i havnt been able to loadup world sinse

full log: http://pastebin.com/wdSjb2KD

---- Minecraft Crash Report ---- // Uh... Did I do that?

Time: 8-12-15 16:09 Description: Ticking block entity

java.lang.StackOverflowError: Ticking block entity at blusunrize.immersiveengineering.common.blocks.metal.TileEntityConnectorLV.transferEnergy(TileEntityConnectorLV.java:188) at blusunrize.immersiveengineering.common.blocks.metal.TileEntityConnectorLV.receiveEnergy(TileEntityConnectorLV.java:165) at cofh.thermalexpansion.block.ender.TileTesseract.receiveEnergy(TileTesseract.java:832) at cofh.thermalexpansion.block.ender.TileTesseract.sendEnergy(TileTesseract.java:293) at cofh.thermalexpansion.block.ender.TileTesseract.receiveEnergy(TileTesseract.java:917) at blusunrize.immersiveengineering.common.blocks.metal.TileEntityConnectorLV.outputEnergy(TileEntityConnectorLV.java:95) at blusunrize.immersiveengineering.common.blocks.metal.TileEntityConnectorLV.transferEnergy(TileEntityConnectorLV.java:205) at blusunrize.immersiveengineering.common.blocks.metal.TileEntityConnectorLV.receiveEnergy(TileEntityConnectorLV.java:165) at cofh.thermalexpansion.block.ender.TileTesseract.receiveEnergy(TileTesseract.java:832) at cofh.thermalexpansion.block.ender.TileTesseract.sendEnergy(TileTesseract.java:284) at cofh.thermalexpansion.block.ender.TileTesseract.receiveEnergy(TileTesseract.java:917)

BluSunrize commented 8 years ago

That's an entirely different error, please create a new issue

purplemonday commented 8 years ago

i did put my log on pastebin! just look at the top where the log starts and youll see the pastebin link ontop

and afraid on the new issue i have no clue what to call it then this exsact topic

On Tue, Dec 8, 2015 at 4:34 PM, BluSunrize notifications@github.com wrote:

That's an entirely different error, please create a new issue. Oh and put your log on pastebin.

— Reply to this email directly or view it on GitHub https://github.com/BluSunrize/ImmersiveEngineering/issues/187#issuecomment-162918183 .

BluSunrize commented 8 years ago

It's also fixed already. I looked into, found they cyclic issue with Tesseracts and fixed it. It's perfectly fine in 0.6.4.

And I edited my message immediately after writing it, because I saw you post the pastebin link. Only you didn't see that because you responded to the email.

purplemonday commented 8 years ago

for information correctness this happend in version 0.6.2 i whasnt able to recreate the issue in 0.6.4 (still testing some more but looks fixed)

BluSunrize commented 8 years ago

Yes. It's fixed. I was able to replicate it after you posted it, so I fixed it for 0.6.3. Sadly, .6.3 also broke the entire transfer system, so .6.4 was made.