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

Species Height Patch #2208

Merged
merged 1 commit into from
Nov 3, 2024
Merged

Conversation

UrPrice
Copy link
Collaborator

@UrPrice UrPrice commented Nov 3, 2024

Описание PR

Этот патч подразумевает следующие изменения:
Для начала, вводятся негласные границы для роста. Возможно они будут в будущем отражены в каком-нибудь ДД, которого нигде нет, но это уже оговорено с Адучем и принято. Уж простите мне мою занятость и невозможность сразу подготовить ДД.

Ограничение для роста расс по скейлу от дефолта (дефолт - хуман, он же base species):
Максимально в высоту: 1,15x
Минимально в высоту: 0,8x

Теперь скейлы между расами будут изменяться на, всегда кратные, 5 сотых единиц (0,05х).
Теперь между расами разница сокращена вдвое, приводя их в более игрово-условный вид, чем ранее.

Из:

Унатх     - 1,15
Человек   - 1
Таяран    - 0,9
Ниан      - 0,8
Вокс      - 0,7

В:

Кто-то    - 1,15
Унатх     - 1,1
Кто-то    - 1,05
Человек   - 1
Таяран    - 0,95
Ниан      - 0,9
Вокс      - 0,85
Дворфы    - 0,8

В новом распределение роста можно заметить относительно нелорную особенность: дворфы ниже воксов по скейлу. С точки зрения механики - это абсолютная правда, однако. Стоит учесть, что дворфы - единственная раса, чей рост меняется не изменением по обеим осям (X и Y), а лишь изменением по оси Y. Поэтому из-за их эффекта приплющенности из-за сжатия по Y - дворфы в любом случае будут создавать эффект более низкого персонажа, нежели другие.

Примечание: однако если разница всё же будет сильно заметна между дворфами и более низкими персонажами, возможно дворфам будет малость увеличен рост в связи с нашими правками.

Помимо роста, также были изменены и хитбоксы некоторых переходящих рас. Теперь унатхи и Арахниды не будут испытывать проблем с прохождением в места с перилами. Более того, теперь эти расы и их позиция стандартного роста - максимальный рост без осложнений. Будущие расы с ростом максимального порога (1,15) - будут иметь хитбокс, вызывающий осложнения в передвижении.

После этого патча, будет создана задача на основе обсуждений ДД роста с Адучем, которая будет нацелена на создание персонализации роста для рас.

Медиа
image
image
image
image

Проверки

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я ознакомился с наставлениями по работе с репозиторием и следовал им при создании PR'а.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения
🆑

  • tweak: подправлен рост рас в более игрово-условный формат!

@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Nov 3, 2024
@@ -46,7 +46,7 @@
!type:PhysShapeCircle
radius: 0.35
# they r smaller
density: 185 #SS220-change-races-height
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

откат к офовским значениям, ибо неясно зачем здесь в принципе были изменения

Copy link

Choose a reason for hiding this comment

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

Дварфов тащить легче, чем воксов (со значениями оффов). Изменения были, т.к. сказали привести плотность дварфов к человеческой.

@stalengd stalengd self-assigned this Nov 3, 2024
@stalengd stalengd merged commit 1bc4814 into SerbiaStrong-220:master Nov 3, 2024
19 checks passed
@UrPrice UrPrice deleted the heigtPatch branch November 4, 2024 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants