Brand new Ballistic Shield#624
Conversation
github-project-automation
Bot
moved this to Требует классификации
in Road to Starship Troopers
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces a new riot shield system featuring knockback resistance, explosion armor bonuses, and mechanics to block pulls and collisions. The review identified several critical issues: logic errors in armor bonus calculations that make passive bonuses unreachable, potential runtime errors in get_readied_shield when called on non-carbon mobs, and incorrect entity checks in the collision logic. Furthermore, the armor calculation lacks clamping, which could lead to negative multipliers in explosion damage, and the knockback resistance reset logic does not account for dual-wielding shields.
половина сделано
в ящике я заменил riot на riot/riot_mp. всё прекрасно, он спавнит новый щит в ящике, но он спавнит ещё и старый, который я вроде бы ЗАМЕНИЛ, ладно...
//SS220 EDIT //SS220 EDIT //SS220 EDIT //SS220 EDIT //SS220 EDIT
конечная на данный момент
|
This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself |
Ingakem
left a comment
Ingakem
left a comment
There was a problem hiding this comment.
Почему вдруг щит магическим образом защищает от действий со всех сторон?
Щиты и так блокируют действия со стороны направления куда смотрит держащий.
Также, сделайте хотя бы немного измененные спрайты, ибо учитывая механы щита, нынешний не особо подходит.
Вот такие пироги.
Co-authored-by: Ingakem <146962612+Ingakem@users.noreply.github.com>
Co-authored-by: Ingakem <146962612+Ingakem@users.noreply.github.com>
Co-authored-by: Ingakem <146962612+Ingakem@users.noreply.github.com>
Co-authored-by: Ingakem <146962612+Ingakem@users.noreply.github.com>
ура
Не знаю что за защита, он работает как обычный щит, стреляют в спину - защиты нет = смерть, всё как с обычным щитом, никакой круговой защиты. как обычный щит, перед, бока. спина, если висит на спине. Да, знаю, потому для нового щита и использую основу в виде уже существующего щита. Спрайты будут. |
Co-authored-by: Ingakem <146962612+Ingakem@users.noreply.github.com>
Co-authored-by: Ingakem <146962612+Ingakem@users.noreply.github.com>
Co-authored-by: Ingakem <146962612+Ingakem@users.noreply.github.com>
|
Не делай вставку моих изменений, ибо они либо полностью удаляют новую строку, либо добавляют с отступом. |
ченджинг бэлэнс оф анти бум армор. словер виз рейзед щилд!!! ловеред анти бум энд пассив блок фор довнед щиелд фор плебеи без навыка
|
This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself |
|
В драфт, пока спрайты рисуют |
mistaces
|
This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself |
3 participants
Что этот PR делает
Добавляет новый баллистический щит. Активировать могут только лица с навыком полиции, после активации, человек что держит щит - получает защиту от толкания спереди и сзади. Полная защита от физического взаимодействия (все интенты) с лицевой стороны + полная защита от граб интента со стороны людей, ксеноморфы спокойно грабают, большая часть абилок игнорирует щит. Дополнительная защита от урона и отталкивания от взрывов, урон всё ещё проходит, зависит от входных данных взрыва. Щит не защищает от стамин-урона, то есть можно оглушить владельца щита продолжительным ПБ дробью, станпродом, дизейблером, резиновыми пулями/гранатами. Щит спокойно пропускает огонь. Изменено снаряжение ЕРТ, которое будет использовать этот щит. Щит добавлен в оружейную брига. Ещё будет изменяться.
Сам запрос, развитие и видео-показы в дискорде: https://discord.com/channels/1097181193939730453/1497226900798570576
Ишшуй: fix #620
Почему это хорошо для игры
Щит является логичным дополнением арсенала МП на таких судах как USS "Алмайер", как для подавления беспорядков, так и для противодействия абордажным командам/применения во время абордажа. Основной щит почти не используется и от него слишком мало толку. Старый нужен был для подавления бунтов, сам бунт должен быть без стрельбы, дабы использовать щит, но к сожалению без стрельбы не обойтись, потому старый щит вообще не подходит под наши реалии.
Изображения изменений
Тестирование
Локалка
Changelog
🆑
add: Добавлен новый баллистический щит
/:cl: