Central Command #20

AwareFoxy · 2024-11-05T23:37:27Z

Описание PR

Rxup/space-station-14#136
Добавляет ЦентКом в игру
Система изначально взята с бэкмена и немного редактирована
Добавляет следующие роли:

Оператор ЦК 2 штука(только вайтлист)

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

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

Технические детали

Медиа

Требования

Я прочитал(а) и следую Рекомендациям по оформлению Pull Request и Changelog.
Я добавил(а) медиафайлы к этому PR или он не требует демонстрации в игре.

Критические изменения

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

add: Теперь на Центральном Командовании могут играть игроки.(По вайтлисту)
tweak: Теперь некоторые шаттлы могут летать на ЦК во время всей смены.

github-actions · 2024-11-05T23:38:05Z

RSI Diff Bot; head commit 6c63a99 merging into 82fdd47
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Interface/Misc/job_icons.rsi

State	Old	New	Status
Nanotrasen			Modified

Edit: diff updated after 6c63a99

lzk228 · 2024-11-06T00:19:36Z

huh

CaptainSqrBeard · 2024-11-06T07:52:51Z

Не очень понимаю в чем геймплей этих ролей. Это же просто максимум 5 человек, оторванных от основного геймплея на станции. Должно быть очень скучно сидеть перекидываться бумагами со станцией, да и при каком нить запросе ERT тоже геймплей будет заключаться лишь в нажатии кнопочки и озвучивании задачи отряда.

И вроде как код добавляет возможность попасть на ЦК при помощи емага, что может сильно повлиять на баланс антагов против экипажа, особенно если на ЦК не будет ролей охраны (а если они по вайтлисту, то их наверняка может и не быть). Да и в целом никто не задумывал, что на ЦК можно попасть не через эвак.

AwareFoxy · 2024-11-06T08:02:30Z

Не очень понимаю в чем геймплей этих ролей. Это же просто максимум 5 человек, оторванных от основного геймплея на станции. Должно быть очень скучно сидеть перекидываться бумагами со станцией, да и при каком нить запросе ERT тоже геймплей будет заключаться лишь в нажатии кнопочки и озвучивании задачи отряда.

И вроде как код добавляет возможность попасть на ЦК при помощи емага, что может сильно повлиять на баланс антагов против экипажа, особенно если на ЦК не будет ролей охраны (а если они по вайтлисту, то их наверняка может и не быть). Да и в целом никто не задумывал, что на ЦК можно попасть не через эвак.

Забыл про емаг. Уберу

AwareFoxy · 2024-11-07T02:23:14Z

Всё пофиксил всё работает

@ksen0morph вам на ревью

kosticia · 2024-11-07T05:45:14Z

Обожаю, когда игроков членят на мелкие группы, которые почти никак друг с другом не пересекаются

NotSoDana · 2024-11-07T16:55:10Z

Предлагаю подождать пока мы в очередной раз переделаем ЦК, а уже потом вводить роли на него

Tornado-Technology · 2024-11-10T09:37:26Z

Учитывая комментарии выше, я бы предложил подождать, учитывая, что в коде есть места, на которые хотелось бы обратить внимание. Ревью будет, но позже, думаю такая задержка не критична, если я ошибаюсь, напишите здесь об этом.

poeMota · 2024-11-12T08:06:59Z

Finally, закрою ПР на основе

CaptainSqrBeard · 2024-11-12T08:43:52Z

Content.Server/Objectives/Systems/KillPersonConditionSystem.cs

+            if (!centcom.Roles.Contains(job.Value.Comp1.JobPrototype.Value))
+            {
+                continue;
+            }


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

Тогда и сам метод желательно переименовать в что-то более общее, типо там FilterByJob(). Хотя не уверен в чём смысл вообще выделять эту часть кода в отдельный метод

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

Тогда и сам метод желательно переименовать в что-то более общее, типо там FilterByJob(). Хотя не уверен в чём смысл вообще выделять эту часть кода в отдельный метод

Не вижу смысла добавлять это кому-либо кроме ЦКшных

CaptainSqrBeard · 2024-11-12T08:49:26Z

Content.Server/_CorvaxNext/Arrivals/CentcommSystem.cs

+            return; // not loaded centcom
+        }
+
+        var shuttleName = "Неизвестный";


Локализация?

CaptainSqrBeard · 2024-11-12T08:49:38Z

Content.Server/_CorvaxNext/Arrivals/CentcommSystem.cs

+        }
+
+        _chat.DispatchStationAnnouncement(CentComGrid,
+            $"Внимание! Радары обнаружили {shuttleName} шаттл, входящий в космическое пространство объекта Центрального Командования!",


Локализация?

CaptainSqrBeard · 2024-11-12T08:55:46Z

Resources/Locale/ru-RU/backmen/actions/centcomm_actions.ftl

В пути к файлу директория бакмена

CaptainSqrBeard · 2024-11-12T08:55:53Z

Resources/Prototypes/Entities/Stations/nanotrasen.yml

+    - BaseStationCargo #Backmen
+    - BaseStationJobsSpawning #Backmen
+    - BaseStationRecords #Backmen
+    - BaseStationShuttles #Backmen


FireNameFN · 2024-11-12T10:14:25Z

Resources/Prototypes/_CorvaxNext/Roles/Jobs/CentCom/centcom_official.yml

+  name: Оператор ЦК
+  description: Оператор Центрального Командования. Истинный бюрократ.


В фтлки.

FireNameFN · 2024-11-12T10:14:59Z

Resources/Prototypes/Maps/centcomm.yml

      stationProto: NanotrasenCentralCommand
      components:
        - type: StationNameSetup
          mapNameTemplate: '{0} Central Command {1}'
          nameGenerator:
            !type:NanotrasenNameGenerator
            prefixCreator: 'TG'
+        - type: StationJobs
+          availableJobs:
+            CCOperator: [ 2, 2]


Suggested change

CCOperator: [ 2, 2]

CCOperator: [2, 2]

FireNameFN · 2024-11-12T10:16:13Z

Resources/Locale/ru-RU/backmen/actions/centcomm_actions.ftl

+centcom-ftl-action-at-centcomm = Вы уже находитесь на ЦентКомме
+centcom-ftl-action-no-station = Станция ЦентКомма не найдена
+centcom-ftl-action-no-pilot = Вы не пилотируете шаттл
+centcom-ftl-action-name = Ключ навигации ЦентКомма
+centcom-ftl-action-description = Позволяет совершить прыжок на станцию ЦентКомма


В русском переводе ЦентКом пишется с одной "м". Вроде бы.

Content.Shared/_CorvaxNext/Shuttles/Components/CentcommShuttleComponent.cs

Content.Shared/_CorvaxNext/CentComm/FtlActionComponent.cs

Content.Shared/_CorvaxNext/CentComm/AllowFtlToCentComComponent.cs

FireNameFN · 2024-11-12T10:17:47Z

Content.Shared/_CorvaxNext/CentComm/AllowFtlToCentComComponent.cs

+
+namespace Content.Shared._CorvaxNext.CentComm;
+
+[RegisterComponent, NetworkedComponent]


Я не уверен, но вроде бы NetworkedComponent лишний, так как нет полей.

Content.Shared/_CorvaxNext/Abilities/CentcomFTLAction.cs

NotSoDana · 2024-11-12T17:05:17Z

Почему всё с двумя "m" написано, кто это придумал бля..

FireNameFN · 2024-11-12T17:06:52Z

Почему всё с двумя "m" написано, кто это придумал

Оффы решили

NotSoDana · 2024-11-12T18:45:11Z

Оффы решили

оффы п...

CaptainSqrBeard · 2024-11-16T14:07:47Z

Мой тестинг

Разное

Криосон не возвращает роль оператора в лейтджоин
Ивенты могут происходить на станции ЦК (как минимум метеориты)
На мапе ЦК спавнятся и обломки в том числе

Ключи перевода без перевода

Нету перевода департамента ЦК (видно при лейтджоине и в должностях персонажа лобби) - department-CentralCommandCorvax
Нету перевода высшей должности (supervisor) оператора ЦК - job-supervisors-centcom2

БСС на мапу ЦК

Неизвестным шаттлом назывался как минимум шаттл ERT, но возможно в целом все шаттлы называются такими (при этом названия у них есть)
Слово "Неизвестный" при объявлении о входе шаттла на мапу надо писать с мелкой буквы.

Проблемы карты

На карте нету криокроватей, предназначенных для простого выхода из игры. Хотя капсулы, используемые для спавна ERT и подобных тоже работают.
На станции ЦК есть факс синдиката, причём исключительно для использования админами. С этими изменениями, доступ к нему получат и простые игроки.
В целом карта может быть не полностью готова под использование простыми игроками.
Атмос на карте не включен автоматом, тоесть операторам ЦК придётся его включать самостоятельно.

Дизайн

Всё ещё уверен насколько геймплей за них будет интересен игрокам.

Я немного опасаюсь возможностей игроков на ЦК, но думаю вайтлист защитит. Вот к что могут сделать игроки ЦК:

Взаимодействовать с эваком и прибытием ЦК, в теории это может быть интересно для игры.
Спавнить ERT (ОБР) и CBURN (РХБЗЗ)
Перемещатся на обычную станцию
Кодов от нюки вроде нету...

Медиа

NotSoDana · 2024-11-16T15:38:06Z

Проблемы карты

На карте нету криокроватей, предназначенных для простого выхода из игры. Хотя капсулы, используемые для спавна ERT и подобных тоже работают.

На станции ЦК есть факс синдиката, причём исключительно для использования админами. С этими изменениями, доступ к нему получат и простые игроки.

В целом карта может быть не полностью готова под использование простыми игроками.

Атмос на карте не включен автоматом, тоесть операторам ЦК придётся его включать самостоятельно.

Как ответственный за СЦК человек заявляю, что на данный момент работаем над очередным реворком. Постараюсь учесть все моменты, однако лично я против удаления факса синди как минимум из-за его полезности. Если игроки зачем-то полезут к нему то это уже работа педалей, но никак не мапперов. На нынешнем цк он достаточно ограничен от зевак, чтобы к нему нельзя было добраться просто так без необходимости и меты.
В остальном, с тезисами соглашусь и поработаю над ними

CaptainSqrBeard · 2024-11-16T18:50:16Z

Как ответственный за СЦК человек заявляю, что на данный момент работаем над очередным реворком. Постараюсь учесть все моменты, однако лично я против удаления факса синди как минимум из-за его полезности. Если игроки зачем-то полезут к нему то это уже работа педалей, но никак не мапперов. На нынешнем цк он достаточно ограничен от зевак, чтобы к нему нельзя было добраться просто так без необходимости и меты. В остальном, с тезисами соглашусь и поработаю над ними

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

NotSoDana · 2024-11-16T19:25:41Z

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

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

CaptainSqrBeard · 2024-11-16T20:28:31Z

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

Тяжело.... Ну ладно

github-actions · 2024-11-24T12:43:18Z

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

Tornado-Technology · 2024-11-29T02:50:21Z

Сделайте нормальную локаль

purr! mreow

2c6a911

AwareFoxy requested a review from Morb0 as a code owner November 5, 2024 23:37

AwareFoxy closed this Nov 5, 2024

github-actions bot added Status: Needs Review Changes: Sprites Changes: Map Изменяет карты Changes: Localization Изменяет локализацию labels Nov 5, 2024

AwareFoxy added 2 commits November 6, 2024 01:56

purr mreoww part 2

f6351e9

:3

05b6514

AwareFoxy changed the title ~~purr! mreow~~ Central Command Nov 6, 2024

AwareFoxy reopened this Nov 6, 2024

AwareFoxy changed the title ~~Central Command~~ [WIP] Central Command Nov 6, 2024

AwareFoxy added 2 commits November 6, 2024 09:06

a bunch of fixes

d503b22

meow!~ fixes

05d7639

AwareFoxy changed the title ~~[WIP] Central Command~~ Central Command Nov 7, 2024

AwareFoxy added the Undergoing Maintainer Discussion Этот PR в настоящее время проходит внутреннее обсуждение мейнтейнеров. label Nov 10, 2024

fixes

4a62c2a

CaptainSqrBeard reviewed Nov 12, 2024

View reviewed changes

FireNameFN reviewed Nov 12, 2024

View reviewed changes

github-actions bot added the S: Merge Conflict У ПРа есть мердж конфликт label Nov 24, 2024

Merge branch 'master' into cc-3

6c63a99

AwareFoxy requested review from Vonsant and Tornado-Technology as code owners November 28, 2024 19:07

github-actions bot added S: Needs Review ПРу необходим ревью size/L and removed S: Merge Conflict У ПРа есть мердж конфликт labels Nov 28, 2024

AwareFoxy closed this Dec 18, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Central Command #20

Central Command #20

AwareFoxy commented Nov 5, 2024 •

edited

Loading

github-actions bot commented Nov 5, 2024 •

edited

Loading

lzk228 commented Nov 6, 2024

CaptainSqrBeard commented Nov 6, 2024

AwareFoxy commented Nov 6, 2024

AwareFoxy commented Nov 7, 2024

kosticia commented Nov 7, 2024

NotSoDana commented Nov 7, 2024

Tornado-Technology commented Nov 10, 2024

poeMota commented Nov 12, 2024

CaptainSqrBeard Nov 12, 2024

AwareFoxy Nov 15, 2024

CaptainSqrBeard Nov 12, 2024

CaptainSqrBeard Nov 12, 2024

CaptainSqrBeard Nov 12, 2024

CaptainSqrBeard Nov 12, 2024

FireNameFN Nov 12, 2024

FireNameFN Nov 12, 2024

FireNameFN Nov 12, 2024

FireNameFN Nov 12, 2024

NotSoDana commented Nov 12, 2024

FireNameFN commented Nov 12, 2024

NotSoDana commented Nov 12, 2024

CaptainSqrBeard commented Nov 16, 2024

NotSoDana commented Nov 16, 2024 •

edited

Loading

Проблемы карты

CaptainSqrBeard commented Nov 16, 2024

NotSoDana commented Nov 16, 2024

CaptainSqrBeard commented Nov 16, 2024

github-actions bot commented Nov 24, 2024

Tornado-Technology commented Nov 29, 2024

		name: Оператор ЦК
		description: Оператор Центрального Командования. Истинный бюрократ.


		namespace Content.Shared._CorvaxNext.CentComm;

		[RegisterComponent, NetworkedComponent]

Central Command #20

Central Command #20

Conversation

AwareFoxy commented Nov 5, 2024 • edited Loading

Описание PR

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

Технические детали

Медиа

Требования

Критические изменения

github-actions bot commented Nov 5, 2024 • edited Loading

lzk228 commented Nov 6, 2024

CaptainSqrBeard commented Nov 6, 2024

AwareFoxy commented Nov 6, 2024

AwareFoxy commented Nov 7, 2024

kosticia commented Nov 7, 2024

NotSoDana commented Nov 7, 2024

Tornado-Technology commented Nov 10, 2024

poeMota commented Nov 12, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

NotSoDana commented Nov 12, 2024

FireNameFN commented Nov 12, 2024

NotSoDana commented Nov 12, 2024

CaptainSqrBeard commented Nov 16, 2024

Мой тестинг

Разное

Ключи перевода без перевода

БСС на мапу ЦК

Проблемы карты

Дизайн

Медиа

NotSoDana commented Nov 16, 2024 • edited Loading

Проблемы карты

CaptainSqrBeard commented Nov 16, 2024

NotSoDana commented Nov 16, 2024

CaptainSqrBeard commented Nov 16, 2024

github-actions bot commented Nov 24, 2024

Tornado-Technology commented Nov 29, 2024

AwareFoxy commented Nov 5, 2024 •

edited

Loading

github-actions bot commented Nov 5, 2024 •

edited

Loading

NotSoDana commented Nov 16, 2024 •

edited

Loading