Every Compat (Stone Zone)

Every Compat (Stone Zone)

74 Downloads

[🐞]: Undetected Stone Types

VoidLeech opened this issue · 16 comments

commented

Code of Conduct

  • Always check you are using the latest version of the mods and its dependencies
  • Ensure the version of EveryCompat & Moonlight Lib are up to date
  • Remove mod that enhances Minecraft: Optifine, Sodium, others. The issue still persists.
  • If you are unsure which mod is the culprit.
    Disable all of your mods and enable them 1-2 mods each time to isolate the culprit
  • Confirm that there is no existing issue with a similar description submitted in the list of issues.

Version - Loader (DO NOT FORGET TO SELECT THE CORRECT VERSION)

1.20.1 - FORGE

StoneZone Version

stonezone-1.20-2.7.2

Moonlight Lib Version

moonlight-1.20-2.13.41

Issue with mods

Create 0.5.1j (create)
Enlightend 5.0.14 (enlightened_end)
Aether Redux 2.0.18 (aether_redux)
Deep Aether 1.0.4 (deep_aether)
Deeper and Darker 1.3.3 (deeperdarker)
Aetherworks Refracted 1.1.2 (aetherworks)
Alex's Caves 2.0.2 (alexscaves)
Rubinated Nether 1.3.0.D (rubinated_nether)

Issue Detail

The following stone types do not generate compat blocks from furniture mods. I recall note block sound mattering for woodgood so I've listed that here as well.

  • create

    • create:limestone (bass drum)
    • create:asurine (bass drum)
    • create:crimsite (bass drum)
    • create:ochrum (bass drum)
    • create:veridium (bass drum)
    • create:scoria (bass drum)
    • create:scorchia (bass drum)
  • enlightened_end

    • enlightened_end:void_shale (bass drum)
  • aether_redux

    • aether_redux:sentrite (harp)
    • aether_redux:driftshale (bass drum)
    • aether_redux:divinite (harp)
  • deep_aether

    • deep_aether:clorite (bass drum)
    • deep_aether:raw_clorite (bass drum)
  • alexscaves

    • alexscaves:radrock (harp)
    • alexscaves:abyssmarine (harp)
    • alexscaves:limestone (harp)
    • alexscaves:guanostone (harp)
    • alexscaves:coprolith (harp)
    • alexscaves:galena (harp)
  • aetherworks

    • aetherworks:suevite (harp)
  • rubinated_nether

    • rubinated_nether:altar_stone (bass drum)
  • deeperdarker

    • deeperdarker:sculk_stone (bass drum)

OPTIONAL: Latest.log | Crash-report Attachment

No response

OPTIONAL: To Produce

No response

commented

partially duplicate of #16?
Also same question as I said there

commented

Oh whoops yeah there’s some overlap. I’ve already listed the note block sounds (and some indeed do not have bass drum)

From what I understand from Xelbayria they also currently need to have a type_bricks (where type is a single word) to be detected, which isn’t the case for these, but they are all stones that also already have some basic variants, which is why I report them as undetected.

commented

I see they have different sounds: HARP. and Most have BASS_DRUM. do they have bricks block?

commented

From the ones I list above, the following do have a bricks block, but they have either harp or two word names xd:

  • aether_redux:sentrite (aether_redux:sentrite_bricks)
  • rubinated_nether:altar_stone (rubinated_nether:altar_stone_bricks)
  • aetherworks:suevite (aetherworks:suevite_bricks)
  • deeperdarker:sculk_stone (deeperdarker:sculk_stone_bricks)
  • enlightened_end:void_shale (enlightened_end:void_shale_bricks)
  • alexscaves:galena (alexscaves:galena_bricks)
  • alexscaves:radrock (alexscaves:radrock_bricks)
  • alexscaves:abyssmarine (alexscaves:abyssmarine_bricks)
  • alexscaves:guanostone (alexscaves:guanostone_bricks)
commented

Two word names separated by _ we will have to add manually. Stuff with (arguably) incorrect note block sounds it's obviously not our issue

commented

I am going to work on that tonight.

commented

Hmm I see. Think it’ll be harder to convince devs they have the wrong sound here than with wood. Have you considered some api so a user/(developer that isn’t the dev of the mod with the wrong sound) can opt-in a stone anyway?

commented

I can manually add these stoneTypes to StoneZone. There are some undetected woodTypes for EveryComp. same method. Even The woodType have the correct sound.

The API is already setup in StoneZone. but for EveryCompat, it's in Moonlight.

commented

Thing is if they are set up wrong it's factually their bug no? API is for Devs, basically by definition. And mod can't depend on configs so no

commented

☝🏻 He's not wrong, tho. They should set these stone to have the correct sound. wood is no different. it's not a lot of works to fix a simple bug, the incorrect sound for stoneTypes

commented

From the ones I list above, the following do have a bricks block, but they have either harp or two word names xd:

* [x]  aether_redux:sentrite (aether_redux:sentrite_bricks)

* [x]  rubinated_nether:altar_stone (rubinated_nether:altar_stone_bricks)

* [x]  aetherworks:suevite (aetherworks:suevite_bricks)

* [x]  deeperdarker:sculk_stone (deeperdarker:sculk_stone_bricks)

* [x]  enlightened_end:void_shale (enlightened_end:void_shale_bricks)

* [x]  alexscaves:galena (alexscaves:galena_bricks)

* [x]  alexscaves:radrock (alexscaves:radrock_bricks)

* [x]  alexscaves:abyssmarine (alexscaves:abyssmarine_bricks)

* [x]  alexscaves:guanostone (alexscaves:guanostone_bricks)

@VoidLeech i've finished add the stonetypes above

is that all? I recommend you to try let DEV know about their sounds and should be change to DRUM_BASS sound

commented

is that all?

The original issue lists more stone types that do have BASS_DRUM, but not a type_bricks (which I don’t think should be disqualifying considering andesite and such don’t have bricks either)

commented

It's safe. That's because simpleFinder is using StoneType.Finder.simple() to find stone and bricks... You can check the log. It won't crash the Minecraft but give error log about not finding StoneTYpe. It's the method used in WoodType and also LeavesType

commented

Is the use of simpleFinder in 5d44d50 safe? Since simpleFinder adds a bricks child when the ones added in this commit don't have bricks.
I'll let the devs know and keep you updated. I know Aether Redux isn't maintained for 1.20.1 anymore but it might still get changed for 1.21.1.

commented

Done. Let me know if DEV won't fix the problem on their side.

commented

ADDED in v2.7.3