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

[1.20.1] [fml mod] No Better End woodtype support #241

Closed
Cixon opened this issue Aug 26, 2023 · 22 comments
Closed

[1.20.1] [fml mod] No Better End woodtype support #241

Cixon opened this issue Aug 26, 2023 · 22 comments
Labels
enhancement New feature or request

Comments

@Cixon
Copy link

Cixon commented Aug 26, 2023

Using Sinytra Connector, FML can easily run fabric mods now. I have Terrestria [fabric mod] installed and their woodtypes are not available to me; same with BetterEnd when using MrCrayfish furniture. However, fabric leaves (hedges) do appear and function great.

Image of BetterEnd Tenanea Hedge running on Forge:

Screenshot (52)

Sinytra connector will be the next big thing so it's definably viable to get the woodtypes working since it appears the fabric leaves work out of box.

Sources:

https://legacy.curseforge.com/minecraft/mc-mods/sinytra-connector
https://legacy.curseforge.com/minecraft/mc-mods/forgified-fabric-api

https://legacy.curseforge.com/minecraft/mc-mods/betterend/files/4677967
(and BCLib, exclude Fabric API)

https://legacy.curseforge.com/minecraft/mc-mods/terrestria/files/4710309
https://legacy.curseforge.com/minecraft/mc-mods/mrcrayfish-furniture-mod/files/4602980

@MehVahdJukaar
Copy link
Owner

That's weird, as far as I knew fabric mods were loaded pretty much same time as others but the static nature of their registration would make their blocks being added immediately, through connector hacks. What's werider is that leaves types work and others do not. Of those are indeed from the same mod it makes even less sense

@MehVahdJukaar
Copy link
Owner

Can you test with other mods that add wood, heck even terrestria on fabric version. Only explanation is that wood detection is somehow failing for those woods

@Cixon
Copy link
Author

Cixon commented Aug 26, 2023

Can you test with other mods that add wood, heck even terrestria on fabric version. Only explanation is that wood detection is somehow failing for those woods

The Terrestria mod doesn’t show any signs of their wood type working with Every Compat, I even tried Cinderscapes. I will say, there’s definitely forge wood type detection as you can see in the screenshot, that mod is gardens of the dead

https://legacy.curseforge.com/minecraft/mc-mods/gardens-of-the-dead

@MehVahdJukaar
Copy link
Owner

So I know for almost certain that terrstria woods and leaves register at same time. With fabric unfortunately there isn't really a dedicated step where that can happen tho but even fabric mod version works under that assumption and these woods do work there so I have absolutely no idea why the registries wouldn't be filled up when forge registers it's stuff

@Cixon
Copy link
Author

Cixon commented Aug 26, 2023

So I know for almost certain that terrstria woods and leaves register at same time. With fabric unfortunately there isn't really a dedicated step where that can happen tho but even fabric mod version works under that assumption and these woods do work there so I have absolutely no idea why the registries wouldn't be filled up when forge registers it's stuff

It could be a Sinytra error, I’d assume. I would @ Su5eD as that’d be our answer to this discrepancy. I’m no programmer so I couldn’t say for sure but it still begs the question as to why leaves show up and wood don’t…

@Cixon
Copy link
Author

Cixon commented Aug 26, 2023

I don’t think it’s open source but here’s github:

https://github.com/Sinytra/ForgifiedFabricAPI

https://github.com/Sinytra/Connector

@MehVahdJukaar
Copy link
Owner

I can't test these days but incase it doesn't work wood types for a couple more fabric mods (and leaves types are registered) the that would confirm tha wood type detection is somehow getting confused. Very weird as it's mostly I'd based and I checked connector loads fabric registries basically as soon as the game starts, way way before forge even loads mods. Confusing stuff . Can you try with twilight forest fabric?

@Cixon
Copy link
Author

Cixon commented Aug 26, 2023

Not available for 1.20.1 fabric

At least on curseforge..maybe there’s a build somewhere

@Cixon
Copy link
Author

Cixon commented Aug 26, 2023

I can test Better Nether and find other fabric wood mods though!

@MehVahdJukaar
Copy link
Owner

Ah hm just trying to think of some other I know really well and I'm sure work on fabirc only. Biomes o plenty?

@Cixon
Copy link
Author

Cixon commented Aug 26, 2023

Ah hm just trying to think of some other I know really well and I'm sure work on fabirc only. Biomes o plenty?

I could attempt to use the fabric version of everycomp woodgood but this may result in the forge compat not working lol

@MehVahdJukaar
Copy link
Owner

Oh lol that would be cursed. Not only that but for how it's made it will not work as it will only see stuff registered by fabric mods

@Cixon
Copy link
Author

Cixon commented Aug 27, 2023

Oh lol that would be cursed. Not only that but for how it's made it will not work as it will only see stuff registered by fabric mods

Made a discovery.....

Architectury API (forge version) ruins the wood registries for fabric mods.
I spent nearly the whole day diagnosing a bunch of things 1 by 1 and found that there's not anything inherently wrong with WoodGood. Disabling shedaniel's API fixed this bizarre situation with only leaves being supported by WoodGood.

It's 50/50 - some mods work, some don't, screenshot below shows Regions Unexplored (fabric)

Screenshot (53)

It appears to be a shedaniel issue but again, I'm no programmer so maybe you could made a patch for the API?
One thing I know for sure is, BetterEnd isn't supported for the current WoodGood FML version.

@Cixon
Copy link
Author

Cixon commented Aug 28, 2023

Conclusion: Some fabric woodtypes work and some need manual support such as BetterEnd and Terrestria.

@Cixon Cixon changed the title [1.20.1] Open WoodGood's Fabric woodtype detection for Forge [1.20.1] [fml mod] No Better End woodtype support Sep 1, 2023
@Xelbayria Xelbayria added the Module Adding mod support to EveryCompat label Oct 13, 2023
@Xelbayria
Copy link
Collaborator

Xelbayria commented Oct 13, 2023

hmm isn't this enhancement for EveryCompat to work with fabric with Minecraft running Forge?
If yes, then perhaps, i should change the label to enhancement instead of module

@MehVahdJukaar
Copy link
Owner

MehVahdJukaar commented Oct 14, 2023

Yeah any issue due to connector can be considered theirs

@Xelbayria Xelbayria added enhancement New feature or request and removed Module Adding mod support to EveryCompat labels Oct 14, 2023
@An-thracene
Copy link

An-thracene commented Oct 17, 2023

the compat for terrestria doesn't work on 1.20 fabric too, is it an expected situation?

@MehVahdJukaar
Copy link
Owner

To be classified as a wood type a block needs to fulfil some characteristic that all wood types have such has having the bass sound type for noteblocks. Do those blocks have that? If not it's not my issue

@An-thracene
Copy link

To be classified as a wood type a block needs to fulfil some characteristic that all wood types have such has having the bass sound type for noteblocks. Do those blocks have that? If not it's not my issue

Well I tested, and I found that it has a harp sound for noteblocks...I will report it to them

@gniftygnome
Copy link

To be classified as a wood type a block needs to fulfil some characteristic that all wood types have such has having the bass sound type for noteblocks.

OK but when you say "some" I wonder what the other assumptions are...

Also, we are talking about the planks block specifically, right?

And finally, we use FabricBlockSettings.copyOf() and I think this issue may be resolved when the following change is merged to FAPI: FabricMC/fabric#3373

@Xelbayria
Copy link
Collaborator

Details:

  • causes of the issues are discovered & reported to Devs of the respective mod
  • an issue caused by Sinytra Connector is not EveryCompat's issue
  • Some mods is somehow screwing up the wood registry on Fabric side

That's it. hope the issue are all resolved.

@MehVahdJukaar
Copy link
Owner

that mod has been blacklisted from ec detection system as they do use dynamic registration too which makes it incompatible with EC since due to many internal issues. It was thus decided to backlist to avoid errors

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

No branches or pull requests

5 participants