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

Better Delta code #12622

Merged
merged 19 commits into from
Apr 29, 2024
Merged

Better Delta code #12622

merged 19 commits into from
Apr 29, 2024

Conversation

BasiaBelov
Copy link
Contributor

@BasiaBelov BasiaBelov commented Dec 17, 2023

Описание изменений

Видева

Ну или кому лень смотреть, что зря, вот мини список изменений:

  • При инициации кода Дельта, освещение меняет свой цвет на красный.
  • За 100 секунд до взрыва запускается сирена.
  • Звук перед взрывом бомбы теперь слышно по всей станции эхом.

Почему и что этот ПР улучшит

Код Дельта теперь станет ДЕЛЬТА! Плюсик к атмосфере.

Авторство

Спасибо этому человеку за сирену - S: Siren by fmagrao | License: Attribution 3.0
А всё закодил и обработал лично я.

Чеинжлог

🆑 Basia

  • add: Улучшена атмосфера кода Дельта.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Sound labels Dec 17, 2023
@TauKitty
Copy link
Contributor

Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности:

https://forum.taucetistation.org/t/fludilka-chast-2/28659/29472

@ZER0-In
Copy link

ZER0-In commented Dec 18, 2023

Аухена, мержим. Сейчас же.

@Kiromaka
Copy link
Contributor

остановка звука есть ? А то там playsound при чеке на armed.. а останавливать звук нечему, пока он не проиграется до конца.

@BasiaBelov
Copy link
Contributor Author

@Kiromaka, я тебя не понял... Какой armed?

@Kiromaka
Copy link
Contributor

@Kiromaka, я тебя не понял... Какой armed?

у тебя музыка стартует если бомба установлена и таймер на 60 секундах, если сразу после этого отрубить бомбу, звук разве не продолжит играть ?

@BasiaBelov
Copy link
Contributor Author

у тебя музыка стартует если бомба установлена и таймер на 60 секундах, если сразу после этого отрубить бомбу, звук разве не продолжит играть ?

Какая музыка? Не добавлял я никаких музыков.

@Kiromaka
Copy link
Contributor

у тебя музыка стартует если бомба установлена и таймер на 60 секундах, если сразу после этого отрубить бомбу, звук разве не продолжит играть ?

Какая музыка? Не добавлял я никаких музыков.

сирена блять

@BasiaBelov
Copy link
Contributor Author

сирена блять

Гармония звуков. Сирена по штучке работает. Так что если разберут бомбу, то она просто перестанет выть, а звук сирены доиграется до конца. Если воткнуть какой-нибудь стопсаунд, что звучать будет ещё уродливей.

Copy link
Contributor

@Chip11-n Chip11-n left a comment

Choose a reason for hiding this comment

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

Пикачуха протестил (из дискорда):

пздц протыкал секунд двадцать ядерку на локалке и уже минут пять слушаю штук тридцать аннонсов

@BasiaBelov
Copy link
Contributor Author

Пикачуха протестил (из дискорда):

пздц протыкал секунд двадцать ядерку на локалке и уже минут пять слушаю штук тридцать аннонсов

Это ПРа не касается. Тут уже просто анонсы полетели триста штук, потому что шитспавн. "Добро пожаловать", "ЦК прислал список подозреваемых", "Активация бомбы", "Код Дельта"

@volas
Copy link
Member

volas commented Dec 21, 2023

sound/machines/nuke_siren.ogg

у нас уже где-то была авиационная сирена в билде

@Chip11-n Chip11-n dismissed their stale review December 22, 2023 18:46

Решение проблемы и неоднократная просьба.

@BasiaBelov
Copy link
Contributor Author

@volas, добавил анти-спам включения и сделал кастомное освещение для Дельты.

@BasiaBelov
Copy link
Contributor Author

Ладно, подожди. Может доделаю...

@BasiaBelov
Copy link
Contributor Author

Ало, "злобный робот", я передумал. Убирай табличку.

@BasiaBelov
Copy link
Contributor Author

Пипец, тиррания...

@BasiaBelov
Copy link
Contributor Author

BasiaBelov commented Mar 4, 2024

@volas, я доделал этот ужас.
Освещение восстанавливается только при переходе из кода Дельта в любой другой.
Кулдаун активации и отключения был переписан в макрос, но сирена нет, так как макрос не поддерживает глобальные таймеры.
В общем... вот да.
image

@BasiaBelov BasiaBelov requested a review from volas March 6, 2024 17:03
@BasiaBelov
Copy link
Contributor Author

Заменил сирену на кое-что пофутуристичней. На мой взгляд, так даже лучше и менее навязчиво.
Потом, перекинул звуки из канала Анонсы в Мастер. Не знаю, почему раньше было так.

@BasiaBelov
Copy link
Contributor Author

Добавил задержку после активации бомбы, перед анонсом кода Дельта.
Свет краснеет, когда закончится анонес о активации бомбы.

@BasiaBelov
Copy link
Contributor Author

BasiaBelov commented Mar 18, 2024

@volas, ещё раз обращусь к вам, указав, что этот ПР существует и почему один фигов кулдаун не через макрос.
Глобальные кулдауны макрос не поддерживает.
P.S. Работает. Просто я дурачок, не знал как это сделать нормально.

Copy link

github-actions bot commented Apr 8, 2024

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@BasiaBelov
Copy link
Contributor Author

Не, брат, ты папутал
Щас всё будет

@BasiaBelov BasiaBelov requested a review from volas April 22, 2024 10:31
@volas volas merged commit 917cf46 into TauCetiStation:master Apr 29, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Sound
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants