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

The Gloves of Silence #7913

Merged
merged 14 commits into from Sep 11, 2021
Merged

The Gloves of Silence #7913

merged 14 commits into from Sep 11, 2021

Conversation

atik1n
Copy link
Member

@atik1n atik1n commented Sep 4, 2021

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

Добавляет перчатки тишины в аплинк за 12ТК.
Данные перчатки почти полностью заглушают звуки в радиусе одного тайла от куклы. Выстрелы, крики, удары, ходьбу. Всё.
Драфт пока что, так как я хочу добавить ещё одни перчатки, как педальные, но я хочу получить уже ревью на код имеющихся.

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

Звуки слышно через пару экранов, может помочь тритоте.

Чеинжлог

🆑 PervertGenius

  • rscadd: Добавлены перчатки тишины

@TauKitty TauKitty added the Feature Новая фича label Sep 4, 2021
@@ -13,6 +13,9 @@ voluminosity = if FALSE, removes the difference between left and right ear.

=======================================================================================================================================*/

/turf
var/sound_coefficient = 1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

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

(Чтобы не вышло так что две перчатки там умножающие на ноль делают что турф навсегда молчит)

Copy link
Contributor

Choose a reason for hiding this comment

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

(Возможно не актуально с тем что перчатки сейчас не умножают этот коэффициент, а прибавляют-убавляют)

@atik1n
Copy link
Member Author

atik1n commented Sep 5, 2021

image

@atik1n atik1n marked this pull request as ready for review September 5, 2021 00:24
Comment on lines +13 to +14
var/bound_width = AM.bound_width + world.icon_size * 2 * _dist
var/bound_height = AM.bound_height + world.icon_size * 2 * _dist
Copy link
Member

Choose a reason for hiding this comment

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

А почему world.icon_size * 2 * _dist тут умножение на 2 по дефолту? Если указать радиус > 1, то у тебя в итоге будет криво значение.

Copy link
Member Author

Choose a reason for hiding this comment

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

тут радиус - это именно размер увеличения границы, а не сам радиус
вот кукла у нас 1х1, радиус 1 = 3х3
есть абстрактный объект 2х1, радиус тишины вокруг 1 = 4х3

Comment on lines +59 to +60
var/list/entered_locs = radius_obj.locs - old_locs
var/list/left_locs = old_locs - radius_obj.locs
Copy link
Member

Choose a reason for hiding this comment

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

смущает, что взаимодействия физических локаций у нас считаются как-то от чисто визуального /obj/effect/overlay

Copy link
Member Author

Choose a reason for hiding this comment

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

Ибо у нас невозможно сделать нормально так, что сами перчатки 1х1, но у них есть "аура", что больше их самих. Данный вариант предложил Киборг и оно работает.

Copy link
Member

Choose a reason for hiding this comment

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

Вроде как, на дистанции рейндж бьонда будет намного медленнее, чем таскание абстрактного радиуса и обычный фор по его элементам.

Я даже пытался это высчитать, но на скриншоте не учтена компонента баунда, которая таскает радиус за собой. Да и код я уже потерял, но там было что-то типо range(9) и такой же большой радиус-предмет.
dreamseeker_M28STjyxB9

Но я щас на этой ветке просто перчатки за собой потаскали не почти ничего не нагрузил.
image

@volas
Copy link
Member

volas commented Sep 5, 2021

Смешанные чувства, на мой взгляд звуки во многом необходимый фидбек для игры, что бы помочь игроку воспринимать игру и всякий экшен и прочие взаимодействия. OOC отчасти.

Но если что-то такое делать, то можно еще посмотреть в сторону текстовых аудио сообщений и их фаллбеков для слепых.

@atik1n
Copy link
Member Author

atik1n commented Sep 5, 2021

Идея родилась из того, что у нас звуки распространяются очень далеко. Именно поэтому звуки из зоны и в зоне этих перчаток будут заглушены. Да, они будут заглушены в том числе и для их носителя из всех источников. То есть в этих перчатках ты не услышишь стрельбу в бриге, если ты на мостике.

var/bound_y = AM.bound_y + world.icon_size * -1 * _dist
coeff = _coeff

radius_obj = new(get_turf(AM))
Copy link
Member

Choose a reason for hiding this comment

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

Я думаю, эту фичу с радиусом можно в еще одну компоненту вынести. Правда, я так и непонял че быстрее, чистый рендж бьонда или /datum/component/bounded который таскает за собой этот радиус.

@KIBORG04 KIBORG04 merged commit 6025748 into TauCetiStation:master Sep 11, 2021
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Sep 11, 2021
volas added a commit that referenced this pull request Sep 11, 2021
@TauKitty
Copy link
Contributor

This pull request has been mentioned on Tau Ceti Station Forum. There might be relevant details there:

https://forum.taucetistation.org/t/fludilka/3304/48432

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants