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

New dry fire sound #34447

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

K-Dynamic
Copy link
Contributor

@K-Dynamic K-Dynamic commented Jan 15, 2025

About the PR

  • New dry fire sound

Why / Balance

Partially fixes #33189.

Uses the much louder dry fire sound from /tg/ station 13 and Delta-V, which audibly alerts both the shooter and nearby players. This reduces reliance on 'counting' bullets or looking at the UI to tell when the gun is empty, which should improve player responsiveness in combat.

Technical details

ogg changes only

Media

dotnet_0g4yRBbwQV.mp4

Requirements

Breaking changes

Changelog

🆑

  • tweak: Empty guns have a louder dry fire sound (i.e. when the chamber or power cell is empty).

@github-actions github-actions bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/XS Denotes a PR that changes 0-9 lines. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Audio Changes: Might require knowledge of audio. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/XS Denotes a PR that changes 0-9 lines. labels Jan 15, 2025
@K-Dynamic
Copy link
Contributor Author

For anyone wanting more examples of in-game audio, Delta-V also uses this sound effect for dry firing.

@@ -12,7 +12,7 @@
"sfrifle_cock.ogg": "https://github.com/discordia-space/CEV-Eris/blob/01f7518e0f8177734a6579aba2bbf76024aa96c4/sound/weapons/guns/interact/sfrifle_cock.ogg"
},
"Empty": {
"empty.ogg": "https://github.com/discordia-space/CEV-Eris/blob/fbde37a8647a82587d363da999a94cf02c2e128c/sound/weapons/guns/misc/gun_empty.ogg",
"empty.ogg": "https://github.com/tgstation/tgstation/blob/d4f678a1772007ff8d7eddd21cf7218c8e07bfc0/sound/weapons/gun_dry_fire.ogg",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not your fault, but what the hell is this attribution format?
This doesn't even mention the licence the file is under and we probably should convert this into an attributions.yml format like we use everywhere else for audio. Not sure if this is a blocker for merging this PR.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, this attribution format seems to have the goal of making it so that individual files can be attributed (how it should be!), instead of the giant, incredibly cursed, one-line "copyright" attributions that are prone to merge conflicts, seen in many of our meta.json's. I think there are... much much better ways of attributing these files though, like what's seen in attribution.yml's.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, and please change the link to the specific commit the file was taken from in case tg changes the audio file at a later point.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, and please change the link to the specific commit the file was taken from in case tg changes the audio file at a later point.

Done

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines. label Jan 16, 2025
@ScarKy0 ScarKy0 added P3: Standard Priority: Default priority for repository items. T: New Feature Type: New feature or content, or extending existing content T: Cleanup Type: Code clean-up, without being a full refactor or feature DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. A: General Interactions Area: General in-game interactions that don't relate to another area. S: Needs Review Status: Requires additional reviews before being fully accepted labels Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: General Interactions Area: General in-game interactions that don't relate to another area. Changes: Audio Changes: Might require knowledge of audio. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. DB: Beginner Friendly Difficulty: Great for beginners. Unambiguous in scope, and explains how to achieve the result. P3: Standard Priority: Default priority for repository items. S: Needs Review Status: Requires additional reviews before being fully accepted size/XS Denotes a PR that changes 0-9 lines. T: Cleanup Type: Code clean-up, without being a full refactor or feature T: New Feature Type: New feature or content, or extending existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Empty guns (ballistics and energy) click sounds are too quiet and need a pop-up
4 participants