Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@monster860
Copy link
Member

@monster860 monster860 commented Jun 5, 2021

Do not merge or testmerge without deploying this website PR first yogstation13/Yogstation.net#2

General Documentation

Intent of your Pull Request

This pull request allows AIs and captains to make voice announcements using their microphones. The way this works is that when they press the button to initiate the voice announcement, a page is opened in a browser that allows them to record a short 30 second voice announcement and upload it to the server. I totally didn't make this PR for the sole purpose of being able to say nya. In all seriousness, I do not play captain or AI very much so you won't be subjected to that.

Here's an example of what it sounds like:

AIs (has a voice filter)
Non-AIs

AI filter and echo effect processing occurs on browser side

Why is this change good for the game?

It allows for players to create creative, immersive announcements.

Wiki Documentation

Briefly describe your PR and the impacts of it, in layman's terms.

It adds the ability for captain or AI to do voice announcements.

What should players be aware of when it comes to the changes your PR is implementing?

Don't abuse this system please.

What general grouping does this PR fall under?

Command I guess

Are there any aspects of the PR that you would like us not to mention on the Wiki?

No

If there are any numerical values involved in your PR that will be relevant to a player, please note them here.

The limit is 30 seconds.

Moderation note

  • There is a limit of 30 seconds to the audio, and the file size is limited to 120 KB. In practice the uploaded files are usually smaller, averaging maybe 20-30 KB. The website records at 20 Kb/s
  • All audio clips are copied to the logs folder when they are played. The person's ckey is part of the filename. The Get Server Logs or Get Current Logs can be used to download the audio clip
  • A CANCEL button is pushed into the chat for admins when the sound clip is played. Pressing this button will immediately stop the sound for everyone.
  • There is a short offset between when admins hear the sound, and when everyone else hears it. This allows admins to cancel the audio before anyone hears an offensive sound clip.
  • A job ban type has been added to allow admins to ban people from this feature.

Changelog

🆑
add: Adds the ability to make voice announcements with a microphone via communications console or AI. Do not abuse this.
/:cl:

@monster860 monster860 requested a review from a team June 5, 2021 10:44
@Yogbot-13 Yogbot-13 added DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Javascript This PR contains javascript code. labels Jun 5, 2021
@github-actions github-actions bot added Admin This PR affects administrators Config Config files need to be changed on the host for this to work tgui This PR affects TGUI labels Jun 5, 2021
@boba247
Copy link

boba247 commented Jun 5, 2021

Is it possible to create a ban specifically for this function?

@monster860
Copy link
Member Author

Is it possible to create a ban specifically for this function?

I am already one step ahead of you, I've already done this. Aaaand I forgot to mention it in the PR body

@Bibby0110
Copy link
Contributor

Bibby0110 commented Jun 5, 2021

this is going to be horrible

i love it

@Jarod1200
Copy link

Is there a preferences toggle that lets you not hear them if you want

@TheLinuxOS
Copy link

Monster you are now my favorite coder

@Marmio64
Copy link
Contributor

Marmio64 commented Jun 5, 2021

Greetings, crew, this is your captain and I would just like to say that I hate all lig- [BWOINK] [BWOINK] [FREEZED] [BANNED]

@alexkar598
Copy link
Member

Not going to lie, this is amazing but I'm not a fan of the idea of voice in ss13

@Hopekz
Copy link
Contributor

Hopekz commented Jun 5, 2021

Good original PR.

Can we maybe get an option to type out text for the recording? So that people can read along like a normal announcement

Not going to lie, this is amazing but I'm not a fan of the idea of voice in ss13

We've had voice in ss13, it was all pre recorded but present nonetheless.

@N3D6
Copy link
Contributor

N3D6 commented Jun 5, 2021

I don't think this is a great idea.
This is 100% gonna be abused, I can't be convinced otherwise.

@Bibby0110
Copy link
Contributor

so voicechat integration next?

@SaltyToastr
Copy link

I cannot express how much of a bad idea this is. we already have the cho*per greifers, now this? Now people will just spam shit in the annoucement system. The argument of "Oh admins will just ban them, duh!" is pretty invalid, as there aren't going to be admins on 24/7.

@TheLinuxOS
Copy link

I cannot express how much of a bad idea this is. we already have the cho*per greifers, now this? Now people will just spam shit in the annoucement system. The argument of "Oh admins will just ban them, duh!" is pretty invalid, as there aren't going to be admins on 24/7.

Admins will hear whatever is being said 3.5 seconds before players do and can cancel it at any time. And you can disable it in prefrences if you dont want to hear it when admins arent on to deal with it. Literally no one can be griefed by this if they dont want to be

var/turf/T = get_turf(M)
if(T.z == z_level)
SEND_SOUND(M, sound2)
sleep(30)
Copy link
Member

Choose a reason for hiding this comment

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

Wouldn't it make more sense to do this?

Suggested change
sleep(30)
sleep(duration + 5 SECONDS)

Copy link
Member Author

Choose a reason for hiding this comment

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

That's an unreasonably long wait time IMO

Copy link
Member

Choose a reason for hiding this comment

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

I'm not too sure about having it played for everyone before admins get to hear the full message tho

Copy link
Contributor

Choose a reason for hiding this comment

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

admins get to hear bad words before everyone else, hit cancel button, sure they'll get a message cut off halfway but whatever right?

@ghost
Copy link

ghost commented Jun 5, 2021

This PR is a terrible idea and I love it, merge this shit.

@maxion12345
Copy link
Contributor

Lots of arguments against this are "oh people could say N word with it!"

you know how many ways you could insert a racial slur into this game? voice annoucements would probably be the most tame of them ngl. I think it's an interesting idea. I, for one, am going to be using it to broadcast shitty text to speech as an AI, it'll be fuckin hiliarious

@Bibby0110
Copy link
Contributor

Bibby0110 commented Jun 13, 2021

Lots of arguments against this are "oh people could say N word with it!"

yeah true it's just a bit different when a 12 year old child drills the N-word into your skull at 10000% volume while you're attempting to stuff the clown down disposals.

But jokes aside this will only really go wrong when there are no admins, the mega-conservative approach would be to just disable this when there are no admins on if you really want to be sure

@N3D6
Copy link
Contributor

N3D6 commented Jun 14, 2021

ngl observing this a bit while testmerged makes it seem like an even worse idea
only thing i've seen it used for so far is unfunny shitposting
if you mute it because you don't wanna hear this stuff if an important announcement is made you're not gonna hear shit of it so if you don't wanna miss something probably important you have to just keep it unmuted and cope with the unfunny
shit idea

@Tipy1802
Copy link

ngl observing this a bit while testmerged makes it seem like an even worse idea
only thing i've seen it used for so far is unfunny shitposting
if you mute it because you don't wanna hear this stuff if an important announcement is made you're not gonna hear shit of it so if you don't wanna miss something probably important you have to just keep it unmuted and cope with the unfunny
shit idea

So the same as AI vox?

@JamieD1 JamieD1 merged commit 7013fa3 into yogstation13:master Jul 1, 2021
Yogbot-13 added a commit that referenced this pull request Jul 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Admin This PR affects administrators Config Config files need to be changed on the host for this to work DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Javascript This PR contains javascript code. tgui This PR affects TGUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.