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

Feature Request: Skyroot Plank lacks the "minecraft:plank" tag. Should it be added? #1822

Open
3 tasks done
ffuentesm opened this issue Oct 20, 2023 · 14 comments
Open
3 tasks done
Assignees
Labels
status/blocked This issue depends on another issue being resolved first. status/dependent This issue requires changes in external API. type/request The issue is a feature or change request and does not fit into the other types. version/1.20.2 This issue affects Minecraft version 1.20.2. version/1.20.4 This issue affects Minecraft version 1.20.4.

Comments

@ffuentesm
Copy link

What Feature Types Apply to This Request?

Block, Item

Other Type

No response

Is This an Enhancement to a Preexisting Feature?

  • I confirm this request is an enhancement to a feature already in The Aether.

What Is Your Feature Request?

I am mostly confused as to why the Skyroot plank lacks the plank tag if the Skyroot Log has the Minecraft:log tag.
image
image

Why Should It Be in the Mod?

For consistency and compatibility with other mods.

Does This Feature Improve Compatibility With Another Mod?

yes, any mod that interact with planks.

Please Read and Confirm the Following

  • I have confirmed this feature is either in the original Aether mod, or improves parity between The Aether and Minecraft or other mods.
  • I have confirmed this suggestion is unique and has not been suggested already.
@ffuentesm ffuentesm added the type/request The issue is a feature or change request and does not fit into the other types. label Oct 20, 2023
@github-project-automation github-project-automation bot moved this to Back Log in The Aether Oct 20, 2023
@bconlon1
Copy link
Contributor

Using the #minecraft:plank tag causes conflicts for Aether recipes like Skyroot Beds and Skyroot Tools, so the plank tag cannot be applied to Skyroot Planks. I would like to write up a NeoForge PR in the future to allow workarounds for conflicts like that, but I have not had the time.

@bconlon1 bconlon1 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 20, 2023
@github-project-automation github-project-automation bot moved this from Back Log to Done in The Aether Oct 20, 2023
@ffuentesm
Copy link
Author

ffuentesm commented Oct 20, 2023

A way to solve that, that I use at least, is the Polymorth mod, which fixes "duplicated recipes" and allows for the user to choose the recipe output. From what I see, moded recipes have priority over the vanilla tags.

I manually added the tag to the skyroot plank and from testing, I see no conflict with the recipes.
image
image
image

As you can see in Polymorph. The Aether recipes take priority over the vanilla recipe outputs.
So even without Polymorth, you get Skyroot items over the vanilla items. And with Polymorph you can then get the vanilla items.
image

@MarioSMB
Copy link

MarioSMB commented Jan 1, 2024

I see this was closed, but it is quite a detriment to modpacks where planks have many uses beyond vanilla and skyroot planks cannot be used in any of those recipes.

As it stands Minecraft does not have wooden variants for blocks like bookshelves and beds, so I do not believe The Aether should either until Minecraft itself does.
I know that might not be a popular opinion as these items have been in the mod for so long but it would solve the conflicts in a vanilla-friendly way.

@Builderdog841
Copy link

Skyroot Tools, Buckets, etc would also have to be removed which are a core part of the mod so that will not happen

@MarioSMB
Copy link

MarioSMB commented Jan 1, 2024

Buckets would not have to be removed as Minecraft does not have wooden buckets.

Does there really need to be a "skyroot stick" that doesn't work with any vanilla or modded recipes? Same could be said for skyroot tools when Minecraft already has wooden tools etc.

Progression should take vanilla into account while complementing it - see the Nether which offers nylium as its wood and blackstone as its stone-replacement, these blocks make regular wooden and stone tools and this isn't a problem.
But I suspect nostalgia will play a big part in these duplicates staying in the mod, for a better or worse experience.

@OzPayn
Copy link
Contributor

OzPayn commented Jan 1, 2024

Skyroot tools have unique abilities, they aren't the same as vanilla wood tools, hence the differing/incompatible recipes/materials. It's not something we'll be changing

@Builderdog841
Copy link

the buckets would also be overwritten by the bowl recipe

@MarioSMB
Copy link

MarioSMB commented Jan 1, 2024

It's an awkward but understandable situation - I only hope some steps can be taken to mitigate the incompatibilities (using tags instead of skyroot sticks directly in recipes for example).
As mentioned by bconlon1 what's really needed is a Forge mechanic to allow these items to be crafted while also including them in the tag, as that would solve the same problem for a plethora of item variants in modded Minecraft.

@ffuentesm
Copy link
Author

Well, I have been playing with a custom datapack that adds the tags to the aether blocks, and so far, no incompatibility has happened.
aether_tweaks.zip

From that I understand, since Aether tools use the item ID for the craft, they take priority over tag recipes.
I also added the milk and water tags to the skyroot buckets. and so far, they work as intended.

Also, in regards to datapacks, there is the polymorph mod that can fix any unforeseen craft incompatibility

@bconlon1 bconlon1 reopened this Jan 1, 2024
@github-project-automation github-project-automation bot moved this from Done to Next Up in The Aether Jan 1, 2024
@ffuentesm
Copy link
Author

the buckets would also be overwritten by the bowl recipe

Not really
image
As you can see, if you add Polymorph you get both options too.
image

@bconlon1 bconlon1 self-assigned this Jan 1, 2024
@ffuentesm
Copy link
Author

Buckets would not have to be removed as Minecraft does not have wooden buckets.

Does there really need to be a "skyroot stick" that doesn't work with any vanilla or modded recipes? Same could be said for skyroot tools when Minecraft already has wooden tools etc.

Progression should take vanilla into account while complementing it - see the Nether which offers nylium as its wood and blackstone as its stone-replacement, these blocks make regular wooden and stone tools and this isn't a problem. But I suspect nostalgia will play a big part in these duplicates staying in the mod, for a better or worse experience.

I feel that the skyroot tools are nice, as they are the efficient tools in the aether and add an air of uniqueness. I still would like the use of tags for mod compatibility as it would allow for an "aether run" without needing to get different types of saplings to craft other mod items. Same reason why I also added the Holystone the minecraft:stone_crafting_materials to it, which allows holystone to be used on block crafting but not the vanilla stone tools. So it can craft things such as pistons, furnaces, dispenses, anything that uses a cobblestone replacement.

But a suggestion that I have to fix this is to just have the recipe use "forge:rods/skywood" or use the item id as the priority.
Like how Immersive Engineering does it with their treated wood items
image

@bconlon1
Copy link
Contributor

bconlon1 commented Jan 4, 2024

Well, I have been playing with a custom datapack that adds the tags to the aether blocks, and so far, no incompatibility has happened. aether_tweaks.zip

From that I understand, since Aether tools use the item ID for the craft, they take priority over tag recipes. I also added the milk and water tags to the skyroot buckets. and so far, they work as intended.

Also, in regards to datapacks, there is the polymorph mod that can fix any unforeseen craft incompatibility

I wonder if because using a custom datapack means youre assumedly loading it with the world startup as opposed to game startup, that somehow causes the Aether recipes to guarantee priority. I have seen a similar situation with addon recipes vs base mod recipes where load stage can determine recipe priority between the two, but hadn't considered if this also applies to the next level down with base mod vs minecraft. If so, I may have to do some testing to investigate this.

@bconlon1
Copy link
Contributor

bconlon1 commented Jan 5, 2024

@ffuentesm I cannot reproduce the functionality of your pack with the Aether.

@bconlon1
Copy link
Contributor

bconlon1 commented Jan 7, 2024

I am going to keep this issue open anyways as a reminder that I need to look into PRing a potential method for getting around this to NeoForge.

@bconlon1 bconlon1 added status/blocked This issue depends on another issue being resolved first. status/dependent This issue requires changes in external API. labels Jan 7, 2024
@bconlon1 bconlon1 added version/1.20.2 This issue affects Minecraft version 1.20.2. version/1.20.4 This issue affects Minecraft version 1.20.4. labels Jan 19, 2024
@bconlon1 bconlon1 added this to the 1.5.0 Update milestone Mar 3, 2024
@bconlon1 bconlon1 moved this from Next Up to Back Log in The Aether Mar 3, 2024
@bconlon1 bconlon1 moved this from Back Log to Next Up in The Aether Apr 2, 2024
@bconlon1 bconlon1 removed this from the 1.5.0 Update milestone May 5, 2024
@bconlon1 bconlon1 added this to the Post-Release Features milestone May 5, 2024
@bconlon1 bconlon1 moved this from Next Up to Back Log in The Aether May 5, 2024
@bconlon1 bconlon1 moved this from Back Log to Next Up in The Aether Jun 18, 2024
@bconlon1 bconlon1 moved this from Next Up to Back Log in The Aether Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/blocked This issue depends on another issue being resolved first. status/dependent This issue requires changes in external API. type/request The issue is a feature or change request and does not fit into the other types. version/1.20.2 This issue affects Minecraft version 1.20.2. version/1.20.4 This issue affects Minecraft version 1.20.4.
Projects
Status: Back Log
Development

No branches or pull requests

5 participants