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

Добавить телепортер Синдиката #35

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

FireNameFN
Copy link
Collaborator

@FireNameFN FireNameFN commented Nov 10, 2024

Описание PR

Добавлен телепортер Синдиката по наработкам @pofitlo-Git. Работает как в Spelunky 2. Телепортирует в свободную клетку по направлению взгляда в дистанции 4-8. Если не находит свободную клетку, гибает. Использовать его можно раз в минуту. Стоит 12 ТК в аплинке.
Оригинал: space-wizards/space-station-14#30176.

Почему / Баланс

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

Ссылка на ветку

https://discord.com/channels/919301044784226385/1304651627260547122

Медиа

Записал бы кто.

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

🆑 FireNameFN, pofitlo-Git

  • add: Добавлен телепортер Синдиката. Стоит 12 ТК.

@github-actions github-actions bot added Changes: Sprites Changes: Localization Изменяет локализацию labels Nov 10, 2024
Copy link

github-actions bot commented Nov 10, 2024

RSI Diff Bot; head commit 3e9c1c1 merging into 3eeb970
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/_CorvaxNext/Effects/teleport_sparks.rsi

State Old New Status
sparks Added

Resources/Textures/_CorvaxNext/Objects/Devices/syndicate_teleporter.rsi

State Old New Status
icon Added

Edit: diff updated after 3e9c1c1

@Seven2280
Copy link

У него проблема есть, экстренная телепортация перестала работать с апстримом.

@FireNameFN
Copy link
Collaborator Author

У него проблема есть, экстренная телепортация перестала работать с апстримом.

Никакой экстренной телепортации. Там написано, как он работает.

Copy link
Collaborator

@Tornado-Technology Tornado-Technology left a comment

Choose a reason for hiding this comment

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

Реализация немного странная, есть ещё ряд других вопросов

[Dependency] private readonly IRobustRandom _random = default!;

[ValidatePrototypeId<EntityPrototype>]
private const string TeleportEffectPrototype = "TeleportEffect";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Поменяй тип на EntProtoId и убери атрибут валидации, это работает именно так

SubscribeLocalEvent<SyndicateTeleporterComponent, UseInHandEvent>(OnUseInHand);
}

private void OnUseInHand(EntityUid entity, SyndicateTeleporterComponent teleproter, UseInHandEvent e)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Лучше используй новую перегрузку OnUseInHand(Entity<SyndicateTeleporterComponent> teleporter, ref UseInHandEvent event)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Для листинга в папке корвакс некст сделай файл по такому же пути


var tile = coordinates.GetTileRef(EntityManager, _map);

if (tile is not null && _turf.IsTileBlocked(tile.Value, CollisionGroup.MobMask))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Колижен группу лучше вынести в компонент


var tile = coordinates.GetTileRef(EntityManager, _map);

if (tile is not null && _turf.IsTileBlocked(tile.Value, CollisionGroup.MobMask))
Copy link
Collaborator

Choose a reason for hiding this comment

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

И да, ты гибнешся при попытке тепа в космос, если тайла нет, то просто добавляй в сейфы

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

tile is null - true => tile is not null - false => if не срабатывает => continue не срабатывает => добавляется в safeCoordinates

@Tornado-Technology
Copy link
Collaborator

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

@FireNameFN
Copy link
Collaborator Author

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

Я просто реализовал идею. Возможно, увеличим цену. Возможно, добавим АПИИ.

@Fildrance
Copy link

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

Забирать шансы у дека это плохо. Возможно размазывать кровяку или просто днк телепортировавшегося после телепорта пофиксит эту проблему??

@FireNameFN FireNameFN marked this pull request as ready for review November 12, 2024 06:49
@FireNameFN FireNameFN requested a review from Morb0 as a code owner November 12, 2024 06:49
@FireNameFN
Copy link
Collaborator Author

Не мёржить. Открыл для тестов.

@FireNameFN FireNameFN marked this pull request as draft November 12, 2024 07:18
@PuroSlavKing
Copy link

Рассказываю лайфхак, называется cherry pick:
WWhiteDreamProject/wwdpublic#63

@Seven2280
Copy link

У него проблема есть, экстренная телепортация перестала работать с апстримом.

Никакой экстренной телепортации. Там написано, как он работает.

Я про то что он сломался и не гибает при попадании в стену.

@FireNameFN
Copy link
Collaborator Author

У него проблема есть, экстренная телепортация перестала работать с апстримом.

Никакой экстренной телепортации. Там написано, как он работает.

Я про то что он сломался и не гибает при попадании в стену.

Система, которую он использует, на девмапе не работает. Проверяй на других картах.

Copy link

github-actions bot commented Dec 9, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the S: Merge Conflict У ПРа есть мердж конфликт label Dec 9, 2024
@FireNameFN
Copy link
Collaborator Author

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Finnaly.

@FireNameFN
Copy link
Collaborator Author

А, не, фигня конфликт.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменяет локализацию Changes: Sprites S: Merge Conflict У ПРа есть мердж конфликт Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants