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

Metal sounds #1334

Closed
wants to merge 2 commits into from
Closed

Metal sounds #1334

wants to merge 2 commits into from

Conversation

sofar
Copy link
Contributor

@sofar sofar commented Oct 20, 2016

2 commits:

  • add default metal node sounds, including
    • digging sound
    • dug sound
    • footstep sound
    • place node sound
  • make all metal nodes use the default metal sounds

@sofar
Copy link
Contributor Author

sofar commented Oct 20, 2016

I intend to use this in the #1318 PR, hence a separate PR

  • fairly trivial
  • will consider alternative sounds (I'm not entirely happy with all of them yet)
  • applied to steel door, trapdoor, steel/copper/bronze blocks and steel ladder and sign

@sofar
Copy link
Contributor Author

sofar commented Oct 20, 2016

  • sound volume (gain) most likely will need adjustment (it's probably too loud?)

@paramat paramat added the Sounds label Oct 20, 2016
@paramat
Copy link
Contributor

paramat commented Oct 20, 2016

I'll try them in-game. Maybe wait for feedback before doing all the volume balancing work.

@paramat
Copy link
Contributor

paramat commented Oct 20, 2016

I think the 'dig' sound should be called 'default_dig_metal' as 'break' suggests 'dug'. For example 'break glass' is the final shattering sound.

@paramat
Copy link
Contributor

paramat commented Oct 20, 2016

Your 'break metal' sound has an unnecessary '.1' in the name.

@sofar
Copy link
Contributor Author

sofar commented Oct 20, 2016

Yes, I really wanted to find another alternative sound sample (have 2-3 as much as possible).

The .1 in the name is entirely fine, it's not like it hurts...

@paramat
Copy link
Contributor

paramat commented Oct 21, 2016

Testing in-game. The footsteps, which i were at first unsure of, are ok at low volume.
The only one i dislike is the dug metal sound, it has a piercing ring to it which i find hard on the ears. I imagine it would be a little harsh with repetition.

@paramat
Copy link
Contributor

paramat commented Oct 21, 2016

Addresses #887

@Fixer-007
Copy link
Contributor

Those are somewhat strange to me:
default_break_metal.1.ogg
default_metal_footstep.1.ogg
default_metal_footstep.2.ogg
default_place_node_metal.1.ogg
default_place_node_metal.2.ogg

@sofar
Copy link
Contributor Author

sofar commented Oct 22, 2016

Yes, these are very crude sound samples. They're mostly just placeholders until someone proposes or finds better sound samples. I wanted to make sure that (1) we'd get a full sound scape implemented for metal nodes (so cover footstep, dig/dug and place) and (2) make sure to use it in all the nodes.

If you have good and usable metal sounds you'd like to propose, please link in this thread.

@Fixer-007
Copy link
Contributor

Hmmmmmmm, maybe freesounds has something useful

@JurajVajda
Copy link

So we are going to have iron, steel and metal now..? :) or is the metal sounds also for nodes like gold, copper, bronze...?

@paramat
Copy link
Contributor

paramat commented Oct 22, 2016

Sounds tend to be difficult to agree on, so i'm trying to not be too fussy. Many of our sounds are humorously crude so let's not be too perfectionist. I might look for an alternative dug sound.

@sofar
Copy link
Contributor Author

sofar commented Oct 23, 2016

@JurajVajda iron is a metal, steel is a metal, gold is a metal, bronze is a metal....

@paramat
Copy link
Contributor

paramat commented Oct 23, 2016

Here's a possible metal break sound https://www.freesound.org/people/JoelAudio/sounds/135465/

@sofar
Copy link
Contributor Author

sofar commented Oct 23, 2016

Entirely redone - updated licenses and origin, used many of the suggestions above.

This is fairly decent now! please try these in-game and check volume levels. I think it's all OK now.

@paramat
Copy link
Contributor

paramat commented Oct 24, 2016

Both dug sounds are too loud. Others seem okay in volume.

These are placeholder sounds. They're not very good just yet,
but they'll help to build out a better set.
@sofar
Copy link
Contributor Author

sofar commented Oct 24, 2016

Lowered dug sounds to gain = 0.5 down from 1.0.

@paramat
Copy link
Contributor

paramat commented Oct 24, 2016

👍

@Fixer-007
Copy link
Contributor

Fixer-007 commented Oct 24, 2016

  • no metal sounds when walking on metal slab/stair (or when placing them)
  • compared dig volume of obsidian vs steel block with pickaxe: obsidian dig has more volume, metal dig sound has little less volume it seems, or maybe it is just obsidian dig bass effect.

Overall sounds seem good to me in game.

@sofar
Copy link
Contributor Author

sofar commented Oct 24, 2016

I'll see why slab/stair for metal nodes isn't getting these sounds, I thought it would be inherited.

@sofar
Copy link
Contributor Author

sofar commented Oct 24, 2016

Fixed metal sounds for stair/slabs for (copper/steel/bronze/gold) block derived versions.

@paramat
Copy link
Contributor

paramat commented Oct 24, 2016

👍

@Fixer-007
Copy link
Contributor

Fixer-007 commented Oct 24, 2016

Seems good now 👍

@paramat
Copy link
Contributor

paramat commented Oct 25, 2016

fa9a345
fbac9be

@paramat paramat closed this Oct 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants