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

Добавляет емаг эвакуационных челноков (подов) #12676

Merged
merged 10 commits into from
Mar 9, 2024

Conversation

simb11
Copy link
Contributor

@simb11 simb11 commented Jan 3, 2024

Описание изменений

Добавляет емаг пода
Делается это кликом емага по интеркому внутри, после этого под прилетает на ЦК с шансом в 100%, а также отстыковывается вместе с шаттлом, даже если шаттл не эвакуационный.

Почему и что этот ПР улучшит

Больше возможностей выполнить цель на побег трейторам.
А ещё убьёт мету с уничтожением багажного отсека в попке шаттла, ведь это будет не единственным способом незаметно улететь.

Авторство

Сделал я, а сама идея - Reallord (если чё все вопросы к нему, с меня чисто тех. часть).

Чеинжлог

🆑 Simbaka

  • add[link]: Добавлена возможность взломать (емагнуть) эвакуационный под.

@TauKitty
Copy link
Contributor

TauKitty commented Jan 3, 2024

Changelog status: ✔️

@TauKitty TauKitty added the Feature Новая фича label Jan 3, 2024
@Chip11-n
Copy link
Contributor

Chip11-n commented Jan 3, 2024

Хочу сразу прояснить момент. Такая возможность известна СБ? Не будет ли потом в БХ мол ууу метагеи!!!

@NinjaPikachuska
Copy link
Contributor

а в чем смысл тогда цели на побег через шаттл?
я думал в этом и задумка что нужно провернуть все так чтобы остаться незамеченным и улететь прямо перед носом сб
мб мету на уничтожение багажного отсека убивать добавлением бессмертия этому багажному отсеку????

@AzzyDreemurr13
Copy link
Contributor

а в чем смысл тогда цели на побег через шаттл?

@NinjaPikachuska напоминаю:
image

Сейчас поды не эффективны для выполнения этой задачи. Так что это уберёт жалобы на не работающие поды с целью

@Fluffy-510
Copy link
Contributor

Чтобы поды не были полноценным аналогом шаттла, можно уменьшить вероятность успешного прилёта со 100 до 75 например
При этом монополия шаттла на побег рушится, но также под не становится его 100% аналогом, так как не будет иметь 100% шанса на успешный отлёт

@AzzyDreemurr13
Copy link
Contributor

Чтобы поды не были полноценным аналогом шаттла, можно уменьшить вероятность успешного прилёта со 100 до 75 например

Тогда зачем в задание сбежать на поде? ◖⁠⚆⁠ᴥ⁠⚆⁠◗ Этот метод должен работать жертвой для емага

@NeOchenLord
Copy link

Хочу сразу прояснить момент. Такая возможность известна СБ? Не будет ли потом в БХ мол ууу метагеи!!!

Так как при этом задействуется емаг, нет, СБ об этой возможности не знает.

@simb11 simb11 requested a review from Chip11-n January 13, 2024 21:46
Copy link
Contributor

@Chip11-n Chip11-n left a comment

Choose a reason for hiding this comment

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

Это через ТМ в любом случае придётся прогонять

code/controllers/subsystem/shuttles.dm Show resolved Hide resolved
code/controllers/subsystem/shuttles.dm Outdated Show resolved Hide resolved
@LudwigVonChesterfield
Copy link
Contributor

что делать если мне не нравится ПР но тут очень крутой рефактор

@AzzyDreemurr13
Copy link
Contributor

AzzyDreemurr13 commented Jan 22, 2024

Уууу так он это делает в шаттл.дм
Готовься устранять конфликты, ибо я там в переводе в шаттл.дм на менял пару вещей (⁠・⁠∀⁠・⁠) (а Пр с переводом уже в тест мерже)

@volas
Copy link
Member

volas commented Jan 29, 2024

Пока блокируется #12624, после мержа вернусь сюда

@simb11
Copy link
Contributor Author

simb11 commented Feb 3, 2024

@volas возвращайся

@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Feb 17, 2024
@F0xiq
Copy link
Contributor

F0xiq commented Feb 19, 2024

Цель "улететь" не засчитывается на поде

@simb11
Copy link
Contributor Author

simb11 commented Feb 19, 2024

вроде теперь должна засчитываться

@volas
Copy link
Member

volas commented Feb 28, 2024

Runtime in code/controllers/subsystem/shuttles.dm:583 : Cannot read null.emagged
  proc name: check emag (/datum/controller/subsystem/shuttle/proc/check_emag)

этот рантайм надо поправить

@volas
Copy link
Member

volas commented Feb 28, 2024

строка отличается, но возможно тут просто бранч не обновлет

@volas volas merged commit b3f6aa0 into TauCetiStation:master Mar 9, 2024
12 checks passed
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Mar 9, 2024
@simb11 simb11 deleted the esc-pods-emag branch March 10, 2024 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Новая фича Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже
Projects
None yet
Development

Successfully merging this pull request may close these issues.