Closing of mesechest does not turn off corresponding mesecons receptor #4809

New Issue

niceride · 2023-06-21T00:28:16Z

niceride commented

2023-06-21 00:28:16 +00:00

Place mesechest. Facing mesechest, left is put pulse, right is take pulse, and rear is open signal. Rear is supposed to also close to turn off that mesecons effector but does not.

Code: https://github.com/minetest-mods/MoreMesecons/blob/master/moremesecons_mesechest/init.lua

Place mesechest. Facing mesechest, left is `put` pulse, right is `take` pulse, and rear is `open` signal. Rear is supposed to also `close` to turn off that mesecons effector but does not. Code: https://github.com/minetest-mods/MoreMesecons/blob/master/moremesecons_mesechest/init.lua

whosit commented

2023-06-21 06:07:05 +00:00

Can confirm that the output on the back stays on forever after opening the chest once.

Seems like this hack stopped working or something on YL conflicts with it:
464699e78b/moremesecons_mesechest/init.lua (L47)
Is it necessary to do it this way? Or it's just a shortcut to write less code? idk

Can confirm that the output on the back stays `on` forever after opening the chest once. Seems like this hack stopped working or something on YL conflicts with it: https://github.com/minetest-mods/MoreMesecons/blob/464699e78b4f1278630ba9b8400d0df6b09975aa/moremesecons_mesechest/init.lua#L47 Is it necessary to do it this way? Or it's just a shortcut to write less code? idk

whosit added the

1. kind/bug

label 2023-06-21 06:08:38 +00:00

whosit commented

2023-06-21 07:24:26 +00:00

That code wraps a function that when serialized will contain "default:chest" string, but we use instrumented code, and it breaks because ours look like this:
return loadstring("\27LJ\2\8R@/yourland/minetest/bin/../builtin/profiler/instrumentation.luay\2\0\9\5\0\0\ ... 7\7time\0measure\0modname\0instrument_name\0func\0start\0\3\8\0\0")

That code wraps a function that when serialized will contain `"default:chest"` string, but we use instrumented code, and it breaks because ours look like this: `return loadstring("\27LJ\2\8R@/yourland/minetest/bin/../builtin/profiler/instrumentation.luay\2\0\9\5\0\0\ ... 7\7time\0measure\0modname\0instrument_name\0func\0start\0\3\8\0\0")`

flux commented

2023-06-21 16:05:18 +00:00

Can confirm that the output on the back stays on forever after opening the chest once.

it's not just that, it stays on forever even if you remove the chest.

That code wraps a function that when serialized will contain "default:chest" string, but we use instrumented code, and it breaks because ours look like this:

yeah that's about where i stopped reading the code and decided that it needs to be cleansed with fire.

> Can confirm that the output on the back stays on forever after opening the chest once. it's not just that, it stays on forever *even if you remove the chest*. > That code wraps a function that when serialized will contain "default:chest" string, but we use instrumented code, and it breaks because ours look like this: yeah that's about where i stopped reading the code and decided that it needs to be cleansed with fire.

👍 2 ❤️ 1 😆 2

tour referenced this issue

2024-04-11 11:37:09 +00:00

flux reports: free chests @ haven are stuck ... #2149

tour commented

2024-04-23 19:54:21 +00:00

https://github.com/minetest-mods/MoreMesecons/pull/35

Sign in to join this conversation.