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

Выбросы не несут вреда #222

Closed
Fahasor opened this issue Jan 8, 2025 · 11 comments · Fixed by #231
Closed

Выбросы не несут вреда #222

Fahasor opened this issue Jan 8, 2025 · 11 comments · Fixed by #231

Comments

@Fahasor
Copy link
Collaborator

Fahasor commented Jan 8, 2025

Description

Собсна выбросы не несут вреда, но визуально он присутствует

Reproduction
Попадите под выброс, досрочно выброс можно вызвать добавив gamerule AnomalyExplosion
Команду не помню, можно подобрать по словам add gamerule

Screenshots

Additional context
Чтобы пофиксить нужно добавить компонент BlowoutTargetComponent к требуемым сущностям, сущности с данным компонентом будут подвержены урону выброса (там есть ещё пару условий, но они довольно логичные, например сущность впринципе должна иметь возможность получать урон), для меня сложность в том, что желательно не трогать сущности оффов, а потому желательно как-то обойти этот момент.

@Fahasor
Copy link
Collaborator Author

Fahasor commented Jan 8, 2025

Возможно как-то можно добавить компонент к ролям, помню что так работают зубы (у всех кроме ренегатов есть контейнер зубов), думаю так можно обойтись меньшей кровью

@HalfElix2
Copy link
Collaborator

Нельзя просто добавить этот компонент людям, крысам и паренту мутантов (или не добавлять, не помню умирают ли они от выброса или нет)? У нас же в игре только они и ходят, не думаю что это будет как-то сложно

@Fahasor
Copy link
Collaborator Author

Fahasor commented Jan 9, 2025

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

Короче добавить прямо в baseHuman не оч хорошо, потому что при всех последующих апстримах это будет ебать мозги тому, кто апстримит.

Может сделать абстракцию baseStslker или типо того? Которая будет от baseHuman наследоватся

@Fahasor
Copy link
Collaborator Author

Fahasor commented Jan 9, 2025

А насчëт крыс, если нужно, то сделаем, как было до апстрима я не помню. В данный момент этого компонента нет ни у кого, а потому крыс сейчас не дамажит

@Fahasor
Copy link
Collaborator Author

Fahasor commented Jan 9, 2025

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

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

Я предлагаю просто дополнить оффовскую сущность, например унаследовать baseHuman, а нашу назвать baseHumanST, и туда запихивать дополнения, т. е. продублировать. Такое решение эквивалентно тому, что мы в baseHuman напишем, но при апстриме триггерится не будет

@SciencePowerCM
Copy link
Collaborator

Кто-то этим щяс занимаеться?

@Fahasor
Copy link
Collaborator Author

Fahasor commented Jan 9, 2025

Думаю нет

Давайте так, если тут никто не отписывается, то значит не занимается

@SciencePowerCM
Copy link
Collaborator

А какая команда для вызозова выброса?

@HalfElix2
Copy link
Collaborator

А какая команда для вызозова выброса?

писали же, gamerule AnomalyExplosion

@HalfElix2
Copy link
Collaborator

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

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

Я предлагаю просто дополнить оффовскую сущность, например унаследовать baseHuman, а нашу назвать baseHumanST, и туда запихивать дополнения, т. е. продублировать. Такое решение эквивалентно тому, что мы в baseHuman напишем, но при апстриме триггерится не будет

Не вижу причин этого не делать. Впринципе можно чисто людей оставить дамажными от выброса, ибо мутанты все таки от пси поля не помирают вроде как, а крысы чисто по приколу будут кайфовать под выбросом)

@SciencePowerCM
Copy link
Collaborator

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

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

Я предлагаю просто дополнить оффовскую сущность, например унаследовать baseHuman, а нашу назвать baseHumanST, и туда запихивать дополнения, т. е. продублировать. Такое решение эквивалентно тому, что мы в baseHuman напишем, но при апстриме триггерится не будет

Может просто, перед апстиримом убрать это и все?

@Fahasor Fahasor linked a pull request Jan 20, 2025 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants