-
Notifications
You must be signed in to change notification settings - Fork 29
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
Builders caused an error, dont know how #41
Comments
Looks like a symptom of what may be an engine bug being discussed at #8067. It appears that trying to read inventory from an unloaded node may fail silently, which would produce these sorts of crashes. I have an idea for a workaround, but it's very inefficient. I'll see if I can come up with a better one, but if I haven't by weekend I'll just bite the bullet and try whatever might work. |
Same here: pandorabox-io/pandorabox.io#113 |
ok, got is also, just since new update this mod... arghh
so can we just declare the nil var inv ? |
Do i get you right, you just want to declare the variable, like so:? local inv See: https://github.com/minetest-mods/digtron/blob/master/util.lua#L314 |
no, I I thnk it need most to be declared about single (= nil) or group (= {}) `tried to fix as most other "attempt to index local '***' (a nil value)" with add before
so using some up in the code, before it is used for fail, add |
after got original digtron files a view days ago ... I didn't use the upper solution now again edit, it also crashes, as there must be some data, ... so is it a new issue, not in an older version ? |
:D
See #5 #16 #17 #22 #30 minetest/minetest#8067 Ideally compile MT with minetest/minetest#8296, reproduce and dump log. Either it will report what exactly goes wrong, or there is something mysterious. |
lol, after the last post is nice, but I will not build with dump option and newest the digtron disapear, but is back when relog ... mhh, I reported, I used last upgrade and maybe I got better back to old not issues version keeps somewhere let me say it clear: I have much bigger other issues then this ! |
Er... sorry but the issue is much older than GreenXenith’s comment, more like 2017 (#5). And it annoys me personally but I couldn’t reproduce it, so had no chance to actually debug it. Moreover, that’s probably not a digtron bug but core bug so fixing it is important; related bugs are reported from time to time over and over again. If you can’t help, that’s sad but not even a reason to comment. Would you read everything, you could find the patch even before commenting, and certainly would find out this issue is much deeper than your naive suggestion. |
I would like to note that Digtrons are getting worse and worse on my server; There are somewhat regular crashes due to this, and now metadata has started shifting without crashing. Digtron meta will get shifted forwards, or parts will get misaligned. Its becoming really annoying to deal with. |
I got a few of those again, but this time with the warning from minetest/minetest#8296 preceeding the actual error:
EDIT: @FaceDeer don't let this discourage you in any way. your mod is awesome :D |
Using the current git HEAD commit , aa927fe I believe I pressed escape while it was building.
|
This occurred on the server this morning. User isn't currently online to ask what builders are on their digtron (but it doesn't appear to be caused by normal building?).
The text was updated successfully, but these errors were encountered: