OreCruncher / BetterRain

Based on the BetterRain mod from Wirsbo
20 stars 12 forks source link

[Enhancement] Mod specific support for Footsteps... #79

Closed OreCruncher closed 6 years ago

OreCruncher commented 8 years ago

This is to track new requests for specific mod support for the Footstep sounds. For mods that are currently supported, and there is a problem with a block sound a new entry needs to be created to track that particular issue. Make sure that the bug information includes the name of the modded block as well as a "should sound like this block in this mod".

Keep in mind that the Footstep logic has fallback sounds based on material type. A request for a mod should be made only if the fallback sounds don't cut it.

Sunconure11 commented 8 years ago

Thaumcraft, Ars Magica 2, Enhanced Biomes, Highlands, Chisel, Various ore mods, and Witchery should be some things you should tackle too.

OreCruncher commented 8 years ago

Are there blocks in those mods that require special support? The ones that have wood/leaves most likely, but the others may not need anything special.

Sunconure11 commented 8 years ago

Possibly various stones, jewels, and also thaumcraft has all that taint stuff. I suggest looking through them to determine what is most fitting.

Et Futurum (grass paths should really have a different sound) could also benefit, as well as Fossils and Archaeology (has wood and leaves)

Sunconure11 commented 8 years ago

Also, this is for vanilla, but, can you make raw clay blocks not sound like stone when stepped on?

Plant Mega Pack might be worth looking at too, for stepping on its various plants.

Harvestcraft and Growthcraft grains should make similar sounds to wheat when stepped on if they are ripe.

OreCruncher commented 8 years ago

Raw clay actually has a sound profile of "sandstone", though it sounds quite similar to the "stone" sound profile. I guess it would depend on whether clay would have a "wet" sound. There is a sound profile called "mud" which sounds a little squishy - you can spawn in sponge blocks to get a sense of what it would sound like.

OreCruncher commented 8 years ago

The clay block has the dirt sound when walked on in Vanilla. I may change it to dirt. I think originally Presence Footsteps had it configured as "concrete" because it looked like concrete and players probably used it as such. But with the different hard clays and things changing it to dirt shouldn't be a problem. (I think "mud" may be a bit too much squishy.)

Sunconure11 commented 8 years ago

The Vegan Option has a jute crop in the latest builds that should play the same sounds that large ferns play when stepped on, IMO.

Sunconure11 commented 8 years ago

Is it possible to detect blocks and assign sounds via oredictionary definitions? That might be an interesting way to knock out many mod requests, and deal with their sounds, as you cannot support every mod ever, you might want to add ways to hook into existing mods and add support there.

OreCruncher commented 8 years ago

In general, yes. Thanks for reminding me. I did similar things in my other mod Thermal Recycling when processing recipes. As a default it would go a long ways.

Sunconure11 commented 8 years ago

I also suggest allowing users to add onto the oredictionary list if you add that option, so unknown oredictionary definitions can be added.

OreCruncher commented 8 years ago

There are quite a number of common ones already - such as the various ores, planks, logs, etc. It would get the vast majority. I would have to think about mod specific ones.

Sunconure11 commented 8 years ago

What I just pushed out as a pull request might add some minor support for AM2, Vegan Option, and a few other mods, including wood stuff. You might want to add a setting for gemstone and obsidian-type blocks though, as I had to put a few under sandstone at the moment.

Picking up mud sounds would help too.

Sunconure11 commented 8 years ago

Don't accept it yet, I am adding some more stuff to it.

Sunconure11 commented 8 years ago

Hopefully this works.

Sunconure11 commented 8 years ago

Okay I think this is safe to merge.

OreCruncher commented 8 years ago

Which ones are obisidian and mud like? There are sound types that have those covered.

Sunconure11 commented 8 years ago

I mean you should add them to the dictionary if you haven't already

Sunconure11 commented 8 years ago

Oh, it seems you have added mud and obsidian-like.

Sunconure11 commented 8 years ago

Also, do blocks such as diamond blocks use a unique sound when stepped on? I'd like to try to conquer some jewel blocks as well.

Sunconure11 commented 8 years ago

I'm assigning some blocks to proper ore dictionary settings at the moment.

OreCruncher commented 8 years ago

The Minecraft defaults for blocks can be found here. The sound type for diamond and emerald blocks is "composite". I would have to set up another entry for that.

Sunconure11 commented 8 years ago

Alright. Expect another pull request with a few AM2 ore dictionary settings shoved into the right entry, and some modded stones added on.

Sunconure11 commented 8 years ago

Also, it appears if I assign an ore to have the proper ore dictionary settings via minetweaker, it does not pick up on the new sounds. Is there anything you can do about that?

OreCruncher commented 8 years ago

Issue with Minetweaker is when it loads. Scripts are executed when a player connects to the world, which is way past the point of when the Footstep mod initializes.

Sunconure11 commented 8 years ago

I see. Is there a way to get around the minetweaker issue? I created some custom ores such as copper using this mod (http://www.curse.com/mc-mods/minecraft/224312-customitems) because I didn't want copper with a billion other things, and registered them via the oredictionary.

Maybe it could also hook into stuff via in game names, as well? Such as say, copper ore that is not under any ore dictionary definition.

Sunconure11 commented 8 years ago

And what about stone bricks and quartz?

OreCruncher commented 8 years ago

Minetweaker - I did something similar with one of my other mods - delaying final init until the last possible moment. I may be able to do something similar for this.

There weren't any ore dictionary entries for stone bricks or quartz. Internally they have sound maps of "brickstone" and "marble" respectively.

Sunconure11 commented 8 years ago

Maybe add entries for them via the oredictionary then, I want to add better support for every mod I can find.

OreCruncher commented 8 years ago

I could add those entries in the Forge mappings, but it would have to depend on what the commonly accepted dictionary names would be for stone bricks. blockQuartz would map to "marble" so I can add that.

Sunconure11 commented 8 years ago

I'll add marble to the proper definition later today or tomorrow. I'll also look for more common dictionary definitions found in various modpacks and mods, as I have done a bit of that to find various definitions.

Sunconure11 commented 8 years ago

Also, can I suggest support for the mod woodstuff? I was able to get most of the rudimentary compatibility for it done via the oredictionary, but, I think the bookshelves need to be handled by you.

OreCruncher commented 8 years ago

Sounds like it would need a specific config unless there is a requirement for having bookshelves being in the ore dictionary.

Sunconure11 commented 8 years ago

Their bookshelves have oredictionary support, but, I meant for stuff like the rustling paper and such.

Sunconure11 commented 8 years ago

Growthcraft 2.5.0 (the preview builds) might need support in the future. I added the cheese entry via the oredictionary for its blocks, and they still sound like stone with this mod installed.

OreCruncher commented 8 years ago

I merged the update after I posted the current BETA on CurseForge.

Sunconure11 commented 8 years ago

Ah, thought something screwed up.

OreCruncher commented 6 years ago

Closing enhancements - 1.7.10 version is in maintenance.