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

Added Ability for Looping Sounds to be heard from further away. #4857

Merged
merged 6 commits into from
Nov 20, 2023
Merged

Added Ability for Looping Sounds to be heard from further away. #4857

merged 6 commits into from
Nov 20, 2023

Conversation

hislittlecuzingames
Copy link
Contributor

@hislittlecuzingames hislittlecuzingames commented Nov 6, 2023

Allows Looping Sounds to be heard from further away.

Explain why it's good for the game

allows for looping alarms to be proximity based and heard from more than 1 room away should one desire.

side note... I was eating pizza and there was hella black ash on the bottom. I think they burned garlic bread. It was nasty.

Testing Photographs and Procedure

screenshot a sound?

okay uhh... I uhh... adjusted settings and played a looping sound and walked away from it.
🆑
code: Added ability to have looping sounds from further away
/:cl:

code/datums/looping_sounds/_looping_sound.dm Outdated Show resolved Hide resolved
code/datums/looping_sounds/_looping_sound.dm Outdated Show resolved Hide resolved
code/datums/looping_sounds/_looping_sound.dm Outdated Show resolved Hide resolved
@Zonespace27 Zonespace27 marked this pull request as draft November 6, 2023 18:00
Copy link
Contributor

@Drulikar Drulikar left a comment

Choose a reason for hiding this comment

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

Additionally, if you're happy with the state of your PR:
image

code/datums/looping_sounds/_looping_sound.dm Show resolved Hide resolved
Thanks to Drulikar for the suggested Documentation.

Co-authored-by: Drathek <[email protected]>
@hislittlecuzingames hislittlecuzingames marked this pull request as ready for review November 8, 2023 19:46
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Nov 16, 2023
@Drulikar Drulikar removed the Stale beg a maintainer to review your PR label Nov 20, 2023
Copy link
Contributor

@Drulikar Drulikar left a comment

Choose a reason for hiding this comment

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

I am uncertain if these changes are actually doing anything. See #4858 (comment) Or at least as configured in that PR I am unable to tell a difference.

@hislittlecuzingames
Copy link
Contributor Author

hislittlecuzingames commented Nov 20, 2023

I am uncertain if these changes are actually doing anything. See #4858 (comment) Or at least as configured in that PR I am unable to tell a difference.

The changes in THIS PR do stuff.
It makes it so sounds can be heard from further away.

Before the added changes, the sounds would be heard from a pre-programmed distance and not further away.

See reply here for videos showing enabled or disabled:
#4858 (comment)

Copy link
Contributor

@Drulikar Drulikar left a comment

Choose a reason for hiding this comment

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

Issue above is resolved; as it is currently, this automerges weirdly with the other PR and it was removing the changes made in this PR. Will have to see if when this is squashed it properly merges the file.

@Drulikar Drulikar added this pull request to the merge queue Nov 20, 2023
Merged via the queue into cmss13-devs:master with commit 986d66f Nov 20, 2023
27 checks passed
cm13-github added a commit that referenced this pull request Nov 20, 2023
@hislittlecuzingames
Copy link
Contributor Author

Issue above is resolved; as it is currently, this automerges weirdly with the other PR and it was removing the changes made in this PR. Will have to see if when this is squashed it properly merges the file.

I've been going to the other PR and then merging THIS branch into the other branch. I may have had moot changes on the other branch that are newer than the changes I made here.

github-merge-queue bot pushed a commit that referenced this pull request Nov 21, 2023
#4858)

…jecting-looping-sounds branch made 5 November 2023 to work)

#ATTENTION!!!!!
Requires this PR to be approved first to work:
#4857
# About the pull request
Makes an alarm the pilots can use on alamo or normandy to tell ground
forces "Hey I plan on taking off soon."

# Explain why it's good for the game

The ship takeoff sound to taking off is like 7 seconds. 
Ground combat is mad intense & people can miss people talking. So having
that alarm may make it so people hear that they're leaving before the
ship leaves.


# Testing Photographs and Procedure
<details>
<summary>Screenshots & Videos</summary>

oh... 

</details>


# Changelog

:cl:
add: Launch Announcement Alarm for dropships to notify ground forces of
departure.
/:cl:

---------

Co-authored-by: Drathek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Improvement Make the code longer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants