Принтер документов #1510

Krosus777 · 2023-10-18T00:17:36Z

Описание PR

Добавлен принтер документов на базе системы протолата
Криминалистические консоли и консоли оповещений заменены на принтер

Медиа

Проверки

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

Изменения

🆑 Krosus

add: Добавлен принтер документов
tweak: Бесполезные консоли заменены на принтеры документов

github-actions · 2023-10-18T00:18:21Z

RSI Diff Bot; head commit 5b7d40d merging into 0ed6a07
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Corvax/Structures/Machines/printer.rsi

State	Old	New	Status
building			Added
icon			Added
idle			Added
inserting			Added
panel			Added

Edit: diff updated after 5b7d40d

into Printer2

SonicHDC · 2023-10-18T01:21:08Z

Всё круто, но замену консолей удаляй, им функционал могут хоть завтра добавить. Да и не везде принтер на месте консоли будет хорошо выглядить

Nimfar11 · 2023-10-18T10:54:25Z

Всё круто, но замену консолей удаляй, им функционал могут хоть завтра добавить. Да и не везде принтер на месте консоли будет хорошо выглядить

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

Krosus777 · 2023-10-18T12:17:40Z

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

Morb0

Для карт есть файл миграция который автоматически заменяет protoId -> protoId на карте при загрузке, поэтому не нужно переделывать консоль.
Пример: https://github.com/space-syndicate/space-station-14/blob/master/Resources/migration.yml#L20

Krosus777 · 2023-10-18T14:44:19Z

Убрал замену консолей
Добавил миграцию консолей

Resources/Prototypes/Corvax/Entities/Objects/Material/Sheets/other.yml

Resources/Prototypes/Corvax/Entities/Structures/Machines/Printer.yml

…ther.yml Co-authored-by: lapatison <[email protected]>

…er.yml Co-authored-by: lapatison <[email protected]>

Resources/Locale/ru-RU/paper/doc-printer.ftl

Resources/Prototypes/Corvax/Entities/Objects/Material/Sheets/other.yml

Morb0 · 2023-10-18T21:07:59Z

Resources/Prototypes/Corvax/Entities/Objects/Material/Sheets/other.yml

@@ -0,0 +1,7 @@
+- type: material
+  id: SheetPrinter


А существующий SheetPaper не подойдет?

А существующий SheetPaper не подойдет?

Нет, так как его только маперы могут добавить, он не спаунится и не заказывается

Resources/Prototypes/Corvax/Entities/Structures/Machines/Printer.yml

Исправил пути

into Printer2

BolloTea · 2023-10-19T10:55:00Z

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

Krosus777 · 2023-10-19T17:32:14Z

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

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

Так как полтикика сервера это поддержание «ванильной» сборки, я специально не делал отдельных плат. Мне кажется чем больше механик к нему накручивать, тем сложнее его будет поддерживать в Корвакс сборке, а значит меньше шансов что он к нам попадет.

Resources/migration.yml

Co-authored-by: Morb <[email protected]>

Morb0

Исправь тесты:

   Multiple failures or warnings in test:
  1)   SheetPrinter 'stack entity' Paper does not have the stack component
  Expected: True
  But was:  False

  2)   SheetPrinter material has no stack prototype
  Expected: True
  But was:  False

lzk228

у ВСЕХ энтити должны быть английские названия, дальше они локализируются через фтл

со спрайтами какая-то хуета, как пример хорошей работы можешь глянуть #1073

lzk228 · 2023-10-22T00:04:50Z

Resources/migration.yml

@@ -8,6 +8,10 @@
 # WallSolid: Window
 # Table: null

+# 2023-10-18 - Corvax


почему оно в середине файла?
должно же по дате сортироваться

Исправлю

lzk228 · 2023-10-22T00:05:57Z

Resources/Prototypes/Corvax/Entities/Structures/Machines/printer.yml

+  parent: BaseLathe
+  id: PrinterDoc
+  name: принтер документов
+  description: Уникальная разработка Nanotrasen.


слишком уныло

Даже не буду комментировать

lzk228 · 2023-10-22T00:06:43Z

Resources/Prototypes/Corvax/Entities/Structures/Machines/printer.yml

@@ -0,0 +1,100 @@
+- type: entity
+  parent: BaseLathe


не проще за основу взять автолат? так как это сделано с принтером одежды

Нет не проще

проще, так как ты будешь использовать в принтере меньше компонентов

lzk228 · 2023-10-22T00:08:27Z

Resources/Textures/Corvax/Structures/Machines/printer.rsi/meta.json

+          0.1,
+          0.1,
+          0.1,
+          2.0


почему финальный кадр целые две секунды?

Так необходимо

lzk228 · 2023-10-22T00:10:25Z

Resources/Textures/Corvax/Structures/Machines/printer.rsi/meta.json

+    },
+    {
+      "name": "building",
+      "delays": [


у тебя каждый рецепт 1 секунду, а стейт идёт 1.8секунд

lzk228 · 2023-10-22T00:11:35Z

Resources/Prototypes/Corvax/Entities/Objects/Materials/Sheets/other.yml

+- type: material
+  id: SheetPrinter
+  stackEntity: Paper
+  name: бумага


а разве названия материалов не должно быть в фтл файле?

Зачем если это для корвакса?

В .yml используй английское название, я потом сгенерирую ftl
или сам можешь с помощью python .\Tools\ss14_ru\yamlextractor.py, только там мусор нагенерит, нужно будет закомитить только новые ftl

lzk228 · 2023-10-22T00:13:22Z

Resources/Prototypes/Corvax/Catalog/Fills/Paper/document.yml

+
+- type: entity
+  parent: Paper
+  id: NOTICEOFLIQUIDATION


почему все буквы заглавные?

Так надо

надо? это называется стандартизация, все айди должны быть в PascalCase

lzk228 · 2023-10-22T00:15:48Z

Resources/Textures/Corvax/Structures/Machines/printer.rsi/building.png

почему тут накладывается фулловый спрайт, можно де накладывать только то, что меняется

Такой спрайт

into Printer2

Morb0

Failed MaterialPrototypeSpawnsStackMaterial [333 ms]
Error Message:
Expected: "SheetPrinter1"
But was: SheetPrinter

lzk228 · 2023-11-27T14:15:20Z

@Krosus777 are you coming back to this?

Krosus777 added 6 commits October 18, 2023 00:18

Printer

7744187

Update Printer.yml

e7c54cb

Update

62bd691

Update other.yml

839a438

Update Printer.yml

02b1d9b

Delete unlit.png

584e71f

Krosus777 requested review from ficcialfaint and Morb0 as code owners October 18, 2023 00:17

Krosus777 added 3 commits October 18, 2023 02:22

Merge branch 'Printer2' of https://github.com/Krosus777/space-station-14

3e60a5d

into Printer2

Update meta.json

cb00d91

name_fix

5714c8c

Morb0 requested changes Oct 18, 2023

View reviewed changes

migration

dc7a7a5

lapatison reviewed Oct 18, 2023

View reviewed changes

Resources/Prototypes/Corvax/Entities/Objects/Material/Sheets/other.yml Outdated Show resolved Hide resolved

lapatison reviewed Oct 18, 2023

View reviewed changes

Resources/Prototypes/Corvax/Entities/Structures/Machines/Printer.yml Outdated Show resolved Hide resolved

Krosus777 and others added 2 commits October 18, 2023 18:41

Update Resources/Prototypes/Corvax/Entities/Objects/Material/Sheets/o…

1d93e76

…ther.yml Co-authored-by: lapatison <[email protected]>

Update Resources/Prototypes/Corvax/Entities/Structures/Machines/Print…

60010c9

…er.yml Co-authored-by: lapatison <[email protected]>

Morb0 requested changes Oct 18, 2023

View reviewed changes

Krosus777 added 3 commits October 18, 2023 23:21

fix

9db1959

Исправил пути

Merge branch 'Printer2' of https://github.com/Krosus777/space-station-14

6184d8a

into Printer2

Rename Printer.yml to printer.yml

8e85129

Morb0 requested changes Oct 19, 2023

View reviewed changes

Resources/migration.yml Outdated Show resolved Hide resolved

Update Resources/migration.yml

d33c629

Co-authored-by: Morb <[email protected]>

Morb0 requested changes Oct 20, 2023

View reviewed changes

lzk228 requested changes Oct 22, 2023

View reviewed changes

Krosus777 added 2 commits October 22, 2023 13:54

FixStack

60db6b1

Merge branch 'Printer2' of https://github.com/Krosus777/space-station-14

5b7d40d

into Printer2

Morb0 reviewed Oct 26, 2023

View reviewed changes

Krosus777 closed this Jan 10, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Принтер документов #1510

Принтер документов #1510

Krosus777 commented Oct 18, 2023

github-actions bot commented Oct 18, 2023 •

edited

Loading

SonicHDC commented Oct 18, 2023

Nimfar11 commented Oct 18, 2023

Krosus777 commented Oct 18, 2023

Morb0 left a comment

Krosus777 commented Oct 18, 2023 •

edited

Loading

Morb0 Oct 18, 2023

Krosus777 Oct 18, 2023

BolloTea commented Oct 19, 2023 •

edited

Loading

Krosus777 commented Oct 19, 2023 •

edited

Loading

Morb0 left a comment

lzk228 left a comment

lzk228 Oct 22, 2023

Krosus777 Oct 22, 2023

lzk228 Oct 22, 2023

Krosus777 Oct 22, 2023

lzk228 Oct 22, 2023

Krosus777 Oct 22, 2023

lzk228 Oct 22, 2023

lzk228 Oct 22, 2023

Krosus777 Oct 22, 2023

lzk228 Oct 22, 2023

lzk228 Oct 22, 2023

Krosus777 Oct 22, 2023

Morb0 Oct 22, 2023

lzk228 Oct 22, 2023

Krosus777 Oct 22, 2023

lzk228 Oct 22, 2023

lzk228 Oct 22, 2023

Krosus777 Oct 22, 2023

Morb0 left a comment

lzk228 commented Nov 27, 2023

+.1,
+.1,
+.1,
+.0

Принтер документов #1510

Принтер документов #1510

Conversation

Krosus777 commented Oct 18, 2023

Описание PR

github-actions bot commented Oct 18, 2023 • edited Loading

SonicHDC commented Oct 18, 2023

Nimfar11 commented Oct 18, 2023

Krosus777 commented Oct 18, 2023

Morb0 left a comment

Choose a reason for hiding this comment

Krosus777 commented Oct 18, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

BolloTea commented Oct 19, 2023 • edited Loading

Krosus777 commented Oct 19, 2023 • edited Loading

Morb0 left a comment

Choose a reason for hiding this comment

lzk228 left a 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

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

Morb0 left a comment

Choose a reason for hiding this comment

lzk228 commented Nov 27, 2023

github-actions bot commented Oct 18, 2023 •

edited

Loading

Krosus777 commented Oct 18, 2023 •

edited

Loading

BolloTea commented Oct 19, 2023 •

edited

Loading

Krosus777 commented Oct 19, 2023 •

edited

Loading