Skip to content

Skrell-M-J#11879

Merged
volas merged 6 commits intoTauCetiStation:masterfrom
Mercurialaste:skrell-foot-4
Aug 12, 2023
Merged

Skrell-M-J#11879
volas merged 6 commits intoTauCetiStation:masterfrom
Mercurialaste:skrell-foot-4

Conversation

@Mercurialaste
Copy link
Contributor

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

Скреллы двигаются немного медленнее людей, с обувью, ещё медленнее.

Будучи без обуви скреллы получают следующие преимущество:
Осколки стекла и тд. не приносят вреда их ногам и рукам.
Мокрый пол, бананы и было - не выводят их из равновесия для подскальзывания.

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

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

При большом уроне мозгу, скреллы больше не матерятся.

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

Больше фич для непопулярной расы.

Авторство

Исполнение: Я, Идея: Энерти.

Чеинжлог

🆑

  • tweak[link]: Добавлена пару фич скреллам.

@TauKitty
Copy link
Contributor

TauKitty commented Aug 9, 2023

Changelog status: ✔️

@TauKitty TauKitty added the Tweak label Aug 9, 2023
@ZER0-In
Copy link

ZER0-In commented Aug 9, 2023

Вроде все звучит интересно, особенно удаление матов при повреждении мозга. Но думаю отсутствие урона от осколков это уже лишнее, не? Типо чего их там такое защищает от осколков.

@Mercurialaste
Copy link
Contributor Author

Вроде все звучит интересно, особенно удаление матов при повреждении мозга. Но думаю отсутствие урона от осколков это уже лишнее, не? Типо чего их там такое защищает от осколков.

что слизнеобразный кожный покров скреллов защищает их подобно улиток

return FLUID_MAX_DEPTH
var/obj/effect/fluid/F = return_fluid()
return (istype(F) ? F.fluid_amount : 0 )
return (depth ? depth : 0) + (F ? (F.fluid_amount ? F.fluid_amount : 0) : 0)
Copy link
Member

Choose a reason for hiding this comment

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

стоит разбить такое сложное условие, сейчас плохо читаемо

return FLUID_MAX_DEPTH
var/obj/effect/fluid/F = return_fluid()
return (istype(F) ? F.fluid_amount : 0 )
return (depth ? depth : 0) + (F ? (F.fluid_amount ? F.fluid_amount : 0) : 0)
Copy link
Member

Choose a reason for hiding this comment

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

вообще такое обновление get_fluid_depth всё же скорее всего что-то ломает. Сходу например - слив проверяет глубину и пытается процессить через get_fluid_depth.

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

На сколько я понимаю, как это все работает, проверка глубины, check_fluid_depth получает значения глубины от get_fluid_depth, которые так или иначе, являются проком тайла, на котором проверяется наличие воды как "эффекта". Я посмотрю как работает слив, но думаю, что, если не ставить слив на тайл с глубиной, (При учете, что сейчас их нельзя никак создать.) ничего плохого не произойдет.

Mercurialaste and others added 2 commits August 11, 2023 01:08
Co-authored-by: Alexander V. <volas@ya.ru>
Comment on lines +133 to -137
if(T && (get_species() != SKRELL || shoes))
tally += T.get_fluid_depth() * 0.0075 // in basic, waterpool have 800 depth
if(T.slowdown)
tally += T.slowdown
var/obj/effect/fluid/F = locate(/obj/effect/fluid) in T
if(F)
tally += F.fluid_amount * 0.005
Copy link
Member

Choose a reason for hiding this comment

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

в связи с чем изменение константы?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

У водных тайлов замедление было = 6, подогнал константу под то, чтобы на тех же водных тайнал, куклы получали все те же 6 единиц замедлений. Могу сделать там более развернутый комментарий, также и у водных тайлов, где использовал не дефайн, а магические числа.

@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Aug 11, 2023
@volas
Copy link
Member

volas commented Aug 12, 2023

боюсь возможны проблемы при столкновении водных тайлов с процессящейся водой, но вода сейчас так себе и требует переработки в любом случае

@volas volas merged commit e1a4064 into TauCetiStation:master Aug 12, 2023
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Aug 12, 2023
@Mercurialaste Mercurialaste deleted the skrell-foot-4 branch October 7, 2023 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже Tweak

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants