Skip to content

Conversation

@notkripton
Copy link
Contributor

@notkripton notkripton commented Aug 2, 2022

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

https://youtu.be/J4COVfevAeU

Список изменений:

  1. Игрок двигается на тайл с объектом когда токает его.
  2. Диагональное передвижение больше не в 2 раза медленнее обычного, а в 1.4 (корень из 2) и будет выглядеть более натурально.
  3. Замедление от пулла мобов теперь зависит от состояния моба. Мобы в сознании и с хелп интентом будут пуллиться быстро, в другом интенте медленнее, мертвым или без сознания еще медленнее.
  4. Уменьшено замедление от пуллинга обжектов.
  5. Замедление от захвата увеличивается с его уровнем и на первом такое же как у пулла.
  6. Контрл+клик по объекту который ты уже пуллишь отменит пулл (хоткей по остановке пулла по кнопке никуда не делся).

UPD:

  1. https://youtu.be/CIqs8yvLNWw
    При столкновении с мобом которого пуллишь пулл не сбрасывается.

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

QoL.

Авторство

Чеинжлог

🆑

  • tweak[link]: Снижено замедление для ходьбы от пулла обьектов. При пулле моба замедление высчитывается от его состояния. Мобы в сознании и с хелп интентом будут пуллиться быстро, в другом интенте медленнее, мертвым или без сознания еще медленнее.
  • add[link]: Теперь игрок двигается на тайл с объектом когда толкает его.
  • add: Замедление от захвата увеличивается с его уровнем и на первом такое же как у пулла.
  • add: Контрл+клик по объекту который ты уже пуллишь отменит пулл.
  • tweak: Диагональное передвижение стало на 30% быстрее.

@TauKitty
Copy link
Contributor

TauKitty commented Aug 2, 2022

Changelog status: ✔️

@TauKitty TauKitty added Feature Новая фича Tweak labels Aug 2, 2022
@Tap0r
Copy link
Contributor

Tap0r commented Aug 2, 2022

Кажется что как-то уж слишком быстро объекты таскаются.
Типо это же ящик с рудой. У него даже колёсиков нету.

@WatlerJe
Copy link
Contributor

WatlerJe commented Aug 2, 2022

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

@notkripton
Copy link
Contributor Author

notkripton commented Aug 2, 2022

Дерганое движение это только на видео

Да, мой комп ведро и не тянет бандикам вместе с локалкой даже в плохом качестве, на серваке будет норм, такие дела)

@KIBORG04
Copy link
Member

KIBORG04 commented Aug 2, 2022

QoL.

Это нихуя не qol

if((direct & (direct - 1)) && mob.loc == n) //moved diagonally successfully
move_delay += add_delay
if((direct & (direct - 1)) && mob.loc == n)
move_delay += add_delay * 0.4 // in general moving diagonally will be 1.4 (sqrt(2)) times slower
Copy link
Member

@KIBORG04 KIBORG04 Aug 2, 2022

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.

Результат налицо даже без микрозамеров (смотреть длину видео)

Замеры
bandicam.2022-08-02.15-59-17-375.mp4
bandicam.2022-08-02.15-59-29-168.mp4

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
Member

Choose a reason for hiding this comment

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

Диагональное движение сейчас абузится чтобы на изи уклониться от ударов и от пуль, теперь же че, это станет почти в 3 раза проще?

Copy link
Contributor Author

@notkripton notkripton Aug 2, 2022

Choose a reason for hiding this comment

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

3 раза проще

На 30 процентов?

В тг (и на гунах?) на такой скорости спокойно живут учитывая что у них прожектайлы еще медленнее наших

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.

Это высчитано по умной формуле и делает диагональное движение справедливее?

Да, это теорема пифагора, по логике вещей такая скорость должна была быть изначально. Вообще мне даже интересно каким образом это должно сломать 'кристальный баланс' робаста таукиты, куклы будут быстрее срезать углы? Как-то неубедительно

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
Member

Choose a reason for hiding this comment

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

Пифагор.... У нас даже по механу кукла делает два шага, а анимация делает диагональное.

Copy link
Member

Choose a reason for hiding this comment

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

Мне кажется, тут изначально математика странная, потому что move_delay != add_delay, там есть еще несколько модификаторов. И этот вариант тоже будет не совсем точным.

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

@TheLone1y
Copy link

Эм я вроде баланс деп так что к чему я это, зачем ускорять на 30% по диагонали если ты на деле делаешь два шага за один клик? Я бы вообще понизил скорость ибо фичи для шарящих, вот сверху была гонка, но ты сделай пробежку по кругу, а не напрямую, в целом переработка пуллов и толканий убирает много рваностей, но это стелсобафф боевки? Добавляй лейбл баланс если хочешь продолжить тему с +30% или убирай эту херню

@necrofill
Copy link

Не согласен с ходьбой по диагонали в 1.4 медленее
Смысл же в том, что ты сокращаешь расстрояние двух тайлов, типо удобство, но скорость та же, будто ты прошел два тайла, нет?
думаю это надо оставить как есть

@notkripton
Copy link
Contributor Author

notkripton commented Aug 2, 2022

Не согласен с ходьбой по диагонали в 1.4 медленее
Смысл же в том, что ты сокращаешь расстрояние двух тайлов, типо удобство, но скорость та же, будто ты прошел два тайла, нет?
думаю это надо оставить как есть

Ты все перепутал, это сейчас диагональками ходят по времени как два тайла, а в пре я ускоряю (или это тонкая шутка и я не понял?)

@necrofill
Copy link

Не согласен с ходьбой по диагонали в 1.4 медленее
Смысл же в том, что ты сокращаешь расстрояние двух тайлов, типо удобство, но скорость та же, будто ты прошел два тайла, нет?
думаю это надо оставить как есть

Ты все перепутал, это сейчас диагональками ходят по времени как два тайла, а в пре я ускоряю (или это тонкая шутка и я не понял?)

Я написал следующее
Сейчас ходьба 2 в раза медленнее по диагонали
Ты хочешь сделать в 1.4
Я считаю, что диагональная ходьба, не дя сокращения дистанции у нас, а для удобства, что б не тыкать вверх лево например
Твоя фишка ломает концепцию тайл за шаг
Я не путал, скорость должна остаться 0.5 по диагонали, если она такая сейчас, и не должна быть ускоренной

@volas volas added the Contentious Спорное изменений требующее обсуждений label Aug 12, 2022
@notkripton
Copy link
Contributor Author

notkripton commented Aug 17, 2022

@volas Можешь сделать заключение по пру? Мейнтейнерам не нравится, но от остальных фидбек положительный. Может сделать какую-нибудь темку на форуме с голосованием?

if((direct & (direct - 1)) && mob.loc == n) //moved diagonally successfully
move_delay += add_delay
if((direct & (direct - 1)) && mob.loc == n)
move_delay += add_delay * 0.4 // in general moving diagonally will be 1.4 (sqrt(2)) times slower
Copy link
Member

Choose a reason for hiding this comment

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

Мне кажется, тут изначально математика странная, потому что move_delay != add_delay, там есть еще несколько модификаторов. И этот вариант тоже будет не совсем точным.

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

@notkripton notkripton requested a review from volas August 30, 2022 10:20
Copy link
Member

@volas volas left a comment

Choose a reason for hiding this comment

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

Просьба еще немного переделать чеинжлог, сами по себе записи сложно понять

if(pulling == AM)
stop_pulling()
step(AM, t)
step(src, t)
Copy link
Member

Choose a reason for hiding this comment

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

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

Comment on lines 195 to 198
if(ismonkey(src))
tally += 1
else if(isslime(src))
tally += 1.5
Copy link
Member

Choose a reason for hiding this comment

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

потерялась проверка на pulling, сейчас это постоянная задержка для обезьян и слаймов

Copy link
Contributor Author

@notkripton notkripton Sep 3, 2022

Choose a reason for hiding this comment

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

Не, эта проверка была у них в мувделее.

	if(pulling)
		tally += count_pull_debuff()

P.S.
Сейчас еще в последнем коммите этот кусок с истайпами на срк убрал и сделал через перегрузки

/mob/living/carbon/monkey/count_pull_debuff()
	return ..() + 1

//Mob pulling
if(ismob(AM))
var/mob/M = AM
tally += M.stat == CONSCIOUS ? ( M.a_intent == INTENT_HELP ? 0 : 0.5 ) : 1
Copy link
Member

Choose a reason for hiding this comment

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

нам давно нужна фича с "советами" в лобби, эти скрытые механики слишком скрытые...

@notkripton
Copy link
Contributor Author

Просьба еще немного переделать чеинжлог, сами по себе записи сложно понять

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

@notkripton notkripton requested a review from volas September 10, 2022 14:03
@notkripton
Copy link
Contributor Author

@volas мерж? 🤐


if(pulling)
tally += count_pull_debuff()
tally += count_pull_debuff()
Copy link
Member

Choose a reason for hiding this comment

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

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

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

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

https://forum.taucetistation.org/t/podskazki-dnya/41407/1

@Deahaka Deahaka mentioned this pull request Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contentious Спорное изменений требующее обсуждений Feature Новая фича Tweak

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants