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

Revert "Random Announcer System (#415)" #555

Closed
wants to merge 1 commit into from

Conversation

VMSolidus
Copy link
Member

Description

This needed a little more time to cook, apparently it wasn't setting an announcer at all in the first place, and genuinely was just picking random sounds from random announcers.

@github-actions github-actions bot added Changes: Audio Changes any ogg files Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files Changes: YML Changes any yml files labels Jul 16, 2024
@DEATHB4DEFEAT
Copy link
Member

I'd rather a fix than revert, what exactly is the problem?

@VMSolidus
Copy link
Member Author

I'd rather a fix than revert, what exactly is the problem?

The problem seems to be(at least from observation) that announcements are not correctly using Fallbacks, and are instead quite literally playing a random announcement from any announcer. The announcer could be Neil, and PirateRadioSpawnRule triggers(which does not, and should never have an announcement), which bafflingly sets off Michael's Code Blue alert. Or seemingly any announcement.

@DEATHB4DEFEAT
Copy link
Member

I'd rather a fix than revert, what exactly is the problem?

The problem seems to be(at least from observation) that announcements are not correctly using Fallbacks, and are instead quite literally playing a random announcement from any announcer. The announcer could be Neil, and PirateRadioSpawnRule triggers(which does not, and should never have an announcement), which bafflingly sets off Michael's Code Blue alert. Or seemingly any announcement.

Weird, no idea how that happened, I'll look into it.

@VMSolidus
Copy link
Member Author

I'd rather a fix than revert, what exactly is the problem?

The problem seems to be(at least from observation) that announcements are not correctly using Fallbacks, and are instead quite literally playing a random announcement from any announcer. The announcer could be Neil, and PirateRadioSpawnRule triggers(which does not, and should never have an announcement), which bafflingly sets off Michael's Code Blue alert. Or seemingly any announcement.

Weird, no idea how that happened, I'll look into it.

I appreciate it

@DEATHB4DEFEAT
Copy link
Member

I'm not having any issues on master, is this a problem with DS?

image

@VMSolidus
Copy link
Member Author

VMSolidus commented Jul 17, 2024

I'm not having any issues on master, is this a problem with DS?

image

It was originally reported by DS, but then I was able to reproduce it locally on an EE Master.

@DEATHB4DEFEAT
Copy link
Member

DEATHB4DEFEAT commented Jul 17, 2024

I'm not having any issues on master, is this a problem with DS?

image

It was originally reported by DS, but then I was able to reproduce it locally on an EE Master.

Could you record the issue?

@DEATHB4DEFEAT
Copy link
Member

Closing this in favor of #557.
Tell me if this should be reopened.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Audio Changes any ogg files Changes: C# Changes any cs files Changes: Localization Changes any ftl files Changes: UI Changes any XAML files Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants