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

Incendary ammo rebalance #2112

Merged
merged 1 commit into from
Oct 27, 2024

Conversation

lexaSvarshik
Copy link

@lexaSvarshik lexaSvarshik commented Oct 18, 2024

Описание PR

Баланс зажигательных патронов, изенение групп урона в соответствии с остальными типами патронов.

Если раньше у них был урон тупой (blunt), сейчас урон уколами (piercing), как и у прочих летальных патронов, имеет смысл.

Дробовым зажигательным патронам отдельно урезан урон, поскольку даже после именения типа урона, они наносили в 1,5 раза больше урона, нежели простые, летальные.

Причины

Проблема тупого (blunt) урона - это не тот тип урона, от которого защищает специальная пуленепробиваемая броня, в связи с чем, патроны тупо игнорировали наличие крутого пуленепробиваемого бронежилета и наносили большой урон. При этом, кукле в противоударной броне (броне от ударов тупыми предметами, типа тулбокса или биты), наносилось куда меньше урона, что совсем не соответствует логике.

Медиа

До ребаланса

Летал .50
imagвыаe
Урон у летала чуть ниже, т.к. не был взят в учёт урон от кровопотери
Зажиги .50
ываыв

После ребаланса

изображение
изображение

Проверки

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

Изменения

🆑 Svarshik

  • tweak: Изменены параметры урона зажигательных патронов

@github-actions github-actions bot added the Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" label Oct 18, 2024
@stalengd stalengd assigned stalengd and Ady4ik and unassigned stalengd Oct 18, 2024
@stalengd stalengd requested a review from Ady4ik October 18, 2024 21:24
@UrPrice UrPrice added the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Oct 18, 2024
@TryHardo7
Copy link

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

Copy link

@Surani1 Surani1 left a comment

Choose a reason for hiding this comment

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

Прикрепи фотографии тестов.

@lexaSvarshik
Copy link
Author

Прикрепи фотографии тестов.

Готово. Тесты были до ТТК апдейта и после, картина одинаковая. Сейчас в шапке скрины тестов уже после ТТК апдейта.

@lexaSvarshik
Copy link
Author

https://discord.com/channels/1097181193939730453/1299411509893402694
Вот в трекере подняли проблему зажиг

@Kirus59 Kirus59 removed the Need to be discussed Требуется обсуждение перед аппрувом/мёрджем label Oct 27, 2024
@Kirus59 Kirus59 merged commit 900c190 into SerbiaStrong-220:master Oct 27, 2024
16 of 17 checks passed
@lexaSvarshik lexaSvarshik deleted the Incendary-rebalance branch October 28, 2024 12:59
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.

7 participants