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

Update stairs / Alias stairs nodes to stairsplus #1935

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

OgelGames
Copy link
Contributor

@OgelGames OgelGames commented Oct 25, 2021

Working on mt-mods/technic#224, I noticed a problem with stairs nodes, they are registered in minetest.after which causes them to have a mod_origin of ??, and also prevents them from being changed by technic_worldgen.

TenPlus1's version doesn't have that problem, so it might be good to switch to that.

EDIT: see below.

@OgelGames OgelGames added the WIP label Oct 25, 2021
@OgelGames OgelGames changed the title Alias stairs nodes to stairsplus / Update stairs Update stairs / Alias stairs nodes to stairsplus Oct 25, 2021
@OgelGames
Copy link
Contributor Author

OgelGames commented Oct 25, 2021

Weird... works for the tests, but not locally...
image
TenPlus1's stairs doesn't check its dependencies correctly, the old (current) version uses minetest.after to get around that, but that causes problems.

EDIT: and adding optional dependencies doesn't work, causes circular dependencies... :/

@OgelGames
Copy link
Contributor Author

OgelGames commented Oct 25, 2021

I think the best solution for this would be to remove the stairs mod, and alias everything to stairsplus nodes (even the nodes that will still be registered by MTG's stairs mod), that should reduce the registered node count significantly, and fix the other problems.

@BuckarooBanzay
Copy link
Contributor

causes them to have a mod_origin of ??

Nice, i always wondered how this was caused

I think the best solution for this would be to remove the stairs mod, and alias everything to stairsplus nodes

Sounds good, i'm still not sure where i got the current stairs mod from, i think i copied it from the forums into a server of mine in the early days and copied it along from world to world until it ended up here :/
Anyway: 👍 for replacing that, will make the recipes a bit clearer too (no duplicates)

Any ideas how to generate an alias list? doing this by hand sounds like a nightmare

@OgelGames
Copy link
Contributor Author

Any ideas how to generate an alias list? doing this by hand sounds like a nightmare

One way I thought of was making a "fake" stairs mod, that instead of registering the nodes, called the stairsplus api, and also aliased the nodes. This would also prevent the recipes for them being added, which is probably a good thing.

I haven't figured out if that is possible yet, or what would happen if the node names were changed (double aliases), but I do know that there are 381 stairs:* nodes currently...

@OgelGames OgelGames marked this pull request as draft January 2, 2022 07:12
@OgelGames
Copy link
Contributor Author

<Huhhila> https://github.com/random-geek/morelights/blob/717553c9cb5faf779dae6df5d9aea2e1674fc242/morelights_extras/init.lua#L181 doesn't appear to be compatible with aliases.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants