Blooper, finally #106
Blooper, finally #106
Conversation
|
что за обрезки скриншотов. |
Какое тебе на проверку? Что именно? |
|
Жду нормального формата PR'а и нормальных доказательств тестирования. |
ghost
closed this
ghost
reopened this
|
вот так бы сразу. уже завтра пройдусь по коду, что то предложу как готовое решение, что то только текстом. |
...encesMenu/preferences/features/character_preferences/the_fluffy_frontier/character_voice.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Yaroslav Nurkov <[email protected]>
Co-authored-by: Yaroslav Nurkov <[email protected]>
Co-authored-by: Yaroslav Nurkov <[email protected]>
ghost
requested a review
from
| addtimer(CALLBACK(src, .proc/bark, listeners, range, vocal_volume, BARK_DO_VARY(vocal_pitch, vocal_pitch_range), vocal_current_bark), total_delay) | ||
| total_delay += rand(DS2TICKS(vocal_speed / BARK_SPEED_BASELINE), DS2TICKS(vocal_speed / BARK_SPEED_BASELINE) + DS2TICKS(vocal_speed / BARK_SPEED_BASELINE)) TICKS | ||
|
|
||
| /randomize_human(mob/living/carbon/human/human) |
There was a problem hiding this comment.
мы ж переопределяем, а не создаём новый
|
да, мой косяк. |
Co-authored-by: ReezeBL <[email protected]>
Co-authored-by: ReezeBL <[email protected]>
Co-authored-by: ReezeBL <[email protected]>
|
Ну и магия. Я смог закоммитить оказывается, чудеса да и только. @RashCat так, те три ревьюшки что я закрыл - ознакомься. На будущее. Это обязательная штука для 515 бьёнда. Что я сделал: убрал ту хрень непонятную с вычитанием. Увеличил радиус слышимости для шёпота на один (до двух, мы ведь видим побитое сообщение, если не очень близко). Поменял константу падения звука. Теперь она... ну... константа. Ещё увеличил радиус барка на 1 до 8 (нуль функции приходится на 8-ую клетку, а не на 7-ую в таком случае) и вуаля. Мы слышим барки на расстоянии всего экрана. Но как громкость меняется и разнится между шёпотом, обычным звуком и криком - мне не нравится. Сам разбирайся, как лучше подобрать. Константу падения 1 не ставь только, хуже будет. |
This reverts commit 2840788.
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may not be viewable. --> <!-- You can view Contributing.MD for a detailed description of the pull request process. --> ## About The Pull Request Ports this: Fluffy-Frontier/FluffySTG#106 Allows for characters to bloop and bark. You can choose a bark for your character and adjust its pitch, range, and speed. You can preview your blooper under Character Preferences. The preference is toggled off by default in Game Preferences. Allows for admins to disable global bloopers Includes Goon assets, which are properly attributed. ## Why It's Good For The Game This isn't a 100% of what I envisioned my original bounty was supposed to do ( #688 ). Although, I believe the players will enjoy having something like it while I tune and adjust this in upcoming changes. This is good for the game because audio cues coming from characters bring a quality of life on the station. This can bring attention to a character if they need attention from others. This allows for more optional customization for a character. ## Proof Of Testing Tested locally with no errors. This may cause errors on live. https://github.com/Bubberstation/Bubberstation/assets/96078776/af28a732-276b-4d0b-8be1-f45b710856b1 ## Changelog <!-- If your PR modifies aspects of the game that can be concretely observed by players or admins you should add a changelog. If your change does NOT meet this description, remove this section. Be sure to properly mark your PRs to prevent unnecessary GBP loss. You can read up on GBP and it's effects on PRs in the tgstation guides for contributors. Please note that maintainers freely reserve the right to remove and add tags should they deem it appropriate. You can attempt to finagle the system all you want, but it's best to shoot for clear communication right off the bat. --> :cl: Shayoki add: Players can now choose a voice for their characters. sound: Added a lot of 'borrowed' sounds to use as voices. admin: Admins can globally disable Bloopers /:cl: <!-- Both :cl:'s are required for the changelog to work! You can put your name to the right of the first :cl: if you want to overwrite your GitHub username as author ingame. --> <!-- You can use multiple of the same prefix (they're only used for the icon ingame) and delete the unneeded ones. Despite some of the tags, changelogs should generally represent how a player might be affected by the changes rather than a summary of the PR's contents. --> <!-- By opening a pull request. You have read and understood the repository rules located on the main README.md on this project. --> --------- Co-authored-by: nevimer <[email protected]> Co-authored-by: lessthanthree <[email protected]> Co-authored-by: Waterpig <[email protected]>
О Pull Request
Спустя огромный промежуток времени я доделал блупер под наш сервер. На этом я не остановлю его поддержку.
Как это может улучшит/повлиять на игровой процесс/ролевую игру
Это довольно интересная тема для обсуждения. Кому-то понравится, кому-то нет. Эта реализация кода подойдёт для каждого. Кто не хочет - не услышит. Кто желает - создаст своему персонажу звучание. Я считаю это клёвым нововведением. С учетом того, что его можно включить/отключить локально.
Z1iQBz1AtL.mp4
Спойлер:
Блупер не слышно через стены. Он никак не поможет обнаружить антага/ помогать обнаружить ерпешек в дорматориях.
dreamseeker_oeGohr9K1i.mp4
Доказательство тестирования к слову вот:
Blooper.mp4
Changelog
🆑
add: our players got ability to make voices for their characters
sound: added big pack of stolen sounds for the blooper
admin: admins now can globally turn off the blooper
/:cl: