Skip to content
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

Sonic screwdriver causes issues with wallmounted, colorwallmounted, etc. #267

Open
3 tasks
wsor4035 opened this issue May 6, 2022 · 5 comments
Open
3 tasks
Labels
Bug Something isn't working Controversial
Milestone

Comments

@wsor4035
Copy link

wsor4035 commented May 6, 2022

see issue mt-mods/homedecor_modpack#33

as @OgelGames calls it, a feature bug, the mtg screwdriver mod has advanced a lot since

-- screwdriver handler code reused from minetest/minetest_game screwdriver @a9ac480
for instance wallmounted and colorwallmounted support have been added https://github.com/minetest/minetest_game/blob/master/mods/screwdriver/init.lua#L68-L83 as well as the on_rotate field. additionally
if not technic.creative_mode then
should check minetest.is_creative_enabled which is per player rather than global

@OgelGames
Copy link
Contributor

Not calling on_rotate is intentional, for the same reason it doesn't call can_dig:

-- contrary to the default screwdriver, do not check for can_dig, to allow rotating machines with CLU's in them

Also the creative check will be fixed by #233

@OgelGames OgelGames changed the title technic sonic screwdriver causes issues with wallmounted, colorwallmounted, etc issues. Sonic screwdriver causes issues with wallmounted, colorwallmounted, etc. May 6, 2022
@wsor4035
Copy link
Author

wsor4035 commented May 6, 2022

Not calling on_rotate is intentional, for the same reason it doesn't call can_dig:

-- contrary to the default screwdriver, do not check for can_dig, to allow rotating machines with CLU's in them

Also the creative check will be fixed by #233

seems easier to add a group to those nodes to bypass, rather than breaking other mods with a on_rotate, specifically ones that disallow

@S-S-X
Copy link
Member

S-S-X commented May 6, 2022

Just yesterday I was thinking about these bugs that have become important features for some players, someone mentioned dropping light nodes somewhere and Technic dropping led light nodes feature came into my mind.

It is terrible bug for some players and useful feature for others so I thought of adding configuration:

drop_light_nodes = true|false

Situation is very similar with this issue (just bit worse results if I've understood issue correctly) so would it be possible to introduce configuration option here?
Any estimates how much extra work and maintenance issues that would generate?

@S-S-X S-S-X added Bug Something isn't working Controversial labels May 6, 2022
@wsor4035
Copy link
Author

wsor4035 commented May 6, 2022

sounds like death by configuration. the more options you add for specific things, the more code and variety of options combined you will have to test to make sure nothing breaks

@S-S-X
Copy link
Member

S-S-X commented May 6, 2022

That's very true but as opinions are very divided it needs decision to either have less options/code while only making some people happy or have more options/code while making more people happy.
First option of course means less maintenance work and second one means more maintenance work.

Why I had this question there:

Any estimates how much extra work and maintenance issues that would generate?

Reason for that question is to determine what it would take to support both sides and if it is really possible at all.
I think it is worth some discussion because during at least past 3 years this issue has popped up multiple times with multiple mods and many wanted to keep behavior adding workarounds to other mods.

(just that "death by configuration" simply means going too far where consequences outweigh benefits, determine if that's the case here)

I do also think it would be better for maintenance and development to change other mods in a way that they would allow doing things that are near impossible without sonic screwdriver but that might actually be a lot more initial work and way harder to get through.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Controversial
Projects
None yet
Development

No branches or pull requests

3 participants