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

Make drones drone #73390

Merged
merged 3 commits into from
May 1, 2024
Merged

Conversation

TheMurderUnicorn
Copy link
Contributor

Summary

Balance "Make Quadcopter Drones Produce Sound"

Purpose of change

Quadcopter style drones did not produce any sound, which is silly because drones make lots of buzzing whirring rotor noises irl.

Describe the solution

I created a new species called ROBOT_FLYING, which is a copy of ROBOT with a few exceptions - they produce WHIRR footsteps and have the LOUDMOVES flag (the only way to give flying creatures footstep sounds), and they don't bleed mechanical fluid (as drones tend to be electric motors with no hydraulic fluid, oil, etc). I assigned this ROBOT_FLYING species to the base_drone, which is what almost all quadcopters copy-from, so effectively hit all small drones with one change. I assigned this species to the camspy as well. It is a huge quadcopter and might contain fluid it should 'bleed', but the tradeoff of no blood but making sound seemed appropriate to me.
image

Describe alternatives you've considered

There was a PR, #65781 that wanted to do this using the PARROT system and have the drones 'bark' sounds the way NPC/Migo/etc have their own barks. But the discussion in that PR suggested it should be done using footsteps so I opted for that.

Testing

Eh, basic testing, made sure they make sounds now, not much to test really.

Additional context

The initial issue for this was #55274 which unfortunately is not truly resolved by this PR only partly addressed. There was also the previous PR that ended up being closed which was #65781. This is not a perfect solution, either, you can't manually assign loudness to footstep sounds, flying creatures by code limitations do not make footsteps, and LOUDMOVES flag is the only way to counteract this which again you can't set that volume manually. So it's not ideal, but I can't make c++ changes and am not even sure what I would change.

I'm not sure if giving this to the camspy was the right call because of its huge-ness, and I did not touch the exodii sniper drone because I have no idea what that is and couldn't tell if it has organic components/I've never seen it before so idk.

Whirring 'footsteps' and no bleeding mechanical fluid, meant for quadcopter esque drones
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON <Bugfix> This is a fix for a bug (or closes open issue) Game: Balance Balancing of (existing) in-game features. json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Apr 29, 2024
@Maleclypse Maleclypse merged commit 53a0a73 into CleverRaven:master May 1, 2024
25 of 26 checks passed
@TheMurderUnicorn TheMurderUnicorn deleted the MakeDronesDrone branch May 1, 2024 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) Game: Balance Balancing of (existing) in-game features. [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants