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

German Accent #945

Closed
Closed

Conversation

DangerRevolution
Copy link
Contributor

@DangerRevolution DangerRevolution commented Mar 8, 2024

About the PR

This accent is a direct 1:1 from YogStation (linked in the .ftl), with minor tweaks. For future proofing and edge-case roleplay scenarios I've kept in references to Antags that aren't here in Station 14.

Gonna be honest, I haven't read the entire list yet.

This PR mostly relied on Bing AI to write this up for me. I'm not insane or a masochist so I didn't write up 400 word combinations for the accent and got Bing to do it for me.

Why / Balance

I've seen a few people over the last week roleplaying Germans, with the accent and all. Thought it'd be cool to make them an accent.

Technical details

1:1 copy of the Scottish accent trait, just for German.

Media

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase
    image

Please note: I'm not German, this is a north German-styled accent and I would appreciate some feedback for words that could be better etc etc.

Changelog DangerRevolution
🆑

  • add: Added the German Accent trait

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files Status: Needs Review labels Mar 8, 2024
Signed-off-by: Danger Revolution! <142105406+DangerRevolution@users.noreply.github.com>
@VMSolidus
Copy link
Contributor

Okay, so. I would really like to apologize for this. I had never actually read the code for Accent traits, and had mistakenly assumed that the Accent traits would have been coded in a sane and reasonable way. In the case of a German accent, it's most easily done by using exactly 7 lines of String.Replace.

You can pretty easily replace Line 24 with about 6 or 7 lines of this specific Dotnet 8.0 function. Luckily Microsoft's own website for Dotnet 8.0 comes with very helpful code examples for how to use the function. In this context, the Variable "message" is a string that you can put directly into String.Replace.

https://learn.microsoft.com/en-us/dotnet/api/system.string.replace?view=net-8.0

Doing so will make the entire combined 800 lines of accent word replacements obsolete. Which if you did so, could cut this PR down from +896 to only +44 to +50 lines.

One more thing, please don't rely on AI to make this translation, in this context it is a genuinely awful idea as it makes the speaker sound like a stereotype of someone with a German accent, which can be seen as fairly offensive. I've went and found an article that describes specific sound shifting techniques for how to portray a realistic German accent, and conveniently enough most of it is done by replacing sounds(letters) with other sounds(letters). This is something that String.Replace can do quite handily.

https://www.backstage.com/magazine/article/how-to-do-a-german-accent-75917/

I'm going to go ahead and draft this PR until these kind of changes are made.

@VMSolidus VMSolidus marked this pull request as draft March 9, 2024 03:40
@VMSolidus VMSolidus mentioned this pull request Mar 9, 2024
1 task
{
var msg = message;

msg = _replacement.ApplyReplacements(msg, "german");
Copy link
Contributor

Choose a reason for hiding this comment

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

This line specifically can be replaced with a couple uses of String.Replace, which would make the Replacements unnecessary entirely.

Copy link
Contributor

Choose a reason for hiding this comment

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

This how accent replacements should be done

@stellar-novas
Copy link
Contributor

One more thing, please don't rely on AI to make this translation, in this context it is a genuinely awful idea as it makes the speaker sound like a stereotype of someone with a German accent,

AI was not used to generate the replacement list, it was used to convert the Yogstation json to ftl.

@VMSolidus
Copy link
Contributor

One more thing, please don't rely on AI to make this translation, in this context it is a genuinely awful idea as it makes the speaker sound like a stereotype of someone with a German accent,

AI was not used to generate the replacement list, it was used to convert the Yogstation json to ftl.

My mistake, however it well.. I'll be honest the SS13 forks aren't exactly known for having sane traits. I very distinctly remember Yoggstation having a Tourettes trait that was literally the worst possible portrayal of Tourettes. I wouldn't consider porting their code to generally be a good idea, as much of it is in extremely poor taste.

@DebugOk
Copy link
Contributor

DebugOk commented Mar 9, 2024

I'm making the decision that no more accents should be added, and I believe this to be too stereotypical. I dislike these sorts of accents in general and think they're of poor taste.

@DebugOk DebugOk closed this Mar 9, 2024
@DangerRevolution DangerRevolution deleted the Accent branch March 31, 2024 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: YML Changes any yml files Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants