X-01 multiphase gun & antique laser gun bullet projectiles #1030
Conversation
WalkthroughВ пул реквесте внесены изменения в описание и функциональность нескольких оружейных сущностей. Добавлена новая мультифазная энергетическая пушка (ent-WeaponMultiphaseEnergyGun) с соответствующими записями в файлах локализации (англ. и рус.). Модифицированы параметры WeaponAntiqueLaser: изменена скорость снаряда, обновлён звук, изменён тип провайдера амуниции, увеличена цена и добавлен компонент BatterySelfRecharger. В файлах hitscan проведены изменения, включая замену типа снаряда и добавление новых сущностей для визуальных и звуковых эффектов, а также обновлены метаданные текстур. Changes
Sequence Diagram(s)sequenceDiagram
participant Игрок
participant Оружие as X-01 Пушка
participant Провайдер as ProjectileBatteryAmmoProvider
participant Звук as SoundGunshot
participant Эффект as BulletImpactEffectRedDisabler
Игрок->>Оружие: Инициирует выстрел
Оружие->>Провайдер: Запрашивает снаряд (BulletEnergyGunLaserMk2)
Провайдер-->>Оружие: Возвращает снаряд
Оружие->>Звук: Воспроизводит звук выстрела
Оружие->>Эффект: Генерирует эффект попадания
sequenceDiagram
participant Игрок
participant Оружие as Antique Laser
participant Зарядка as BatterySelfRecharger
participant Провайдер as ProjectileBatteryAmmoProvider
participant Звук as SoundGunshot
Игрок->>Оружие: Инициирует выстрел
Оружие->>Зарядка: Проверяет уровень заряда
Зарядка-->>Оружие: Подтверждает заряд
Оружие->>Провайдер: Запрашивает снаряд (BulletEnergyGunLaserMk2)
Провайдер-->>Оружие: Передаёт снаряд
Оружие->>Звук: Воспроизводит звук выстрела
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
github-actions
bot
added
size/M
Changes: Sprites
|
RSI Diff Bot; head commit 444e003 merging into 9341b8b Resources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi
Resources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi
|
Я не тупой!!!
Agronomist-NN
changed the title
| soundGunshot: | ||
| path: /Audio/Weapons/Guns/Gunshots/laser_cannon.ogg |
| @@ -596,7 +599,7 @@ | |||
| - Sidearm | |||
| - WeaponAntiqueLaser | |||
| - type: StaticPrice | |||
| price: 750 | |||
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (3)
Resources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/meta.json (1)
4-4: Исправьте опечатку в информации об авторских правах.В слове "spriyed" допущена опечатка.
- "copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/484fcb08e8e2058b81059424fb5aa77034a3fcf1. Multiphase variant spriyed by Agronomist", + "copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/484fcb08e8e2058b81059424fb5aa77034a3fcf1. Multiphase variant sprited by Agronomist",Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml (2)
44-44: Удаление лишних пробелов
Обнаружены завершающие лишние пробелы в строке 44 – удалите их, чтобы соответствовать стандартам форматирования YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 44-44: trailing spaces
(trailing-spaces)
107-107: Удаление лишних пробелов
Найдены завершающие лишние пробелы на строке 107 – уберите их для корректного форматирования.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 107-107: trailing spaces
(trailing-spaces)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (26)
Resources/Audio/Backmen/Weapons/Energy/Resources_Audio_DeltaV_Weapons_Guns_Gunshots_laser.oggis excluded by!**/*.oggResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/base.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/equipped-SUITSTORAGE.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/icon.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-left-0.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-left-1.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-left-2.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-left-3.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-right-0.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-right-1.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-right-2.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-right-3.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/mag-unshaded-0.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/mag-unshaded-1.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/mag-unshaded-2.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/mag-unshaded-3.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/mag-unshaded-4.pngis excluded by!**/*.pngResources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/base.pngis excluded by!**/*.pngResources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/icon.pngis excluded by!**/*.pngResources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/mag-unshaded-0.pngis excluded by!**/*.pngResources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/mag-unshaded-1.pngis excluded by!**/*.pngResources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/mag-unshaded-2.pngis excluded by!**/*.pngResources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/mag-unshaded-3.pngis excluded by!**/*.pngResources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/mag-unshaded-4.pngis excluded by!**/*.png
📒 Files selected for processing (6)
Resources/Locale/en-US/ss14-ru/prototypes/entities/objects/weapons/guns/battery/battery_guns.ftl(1 hunks)Resources/Locale/ru-RU/ss14-ru/prototypes/entities/objects/weapons/guns/battery/battery_guns.ftl(1 hunks)Resources/Prototypes/Entities/Objects/Weapons/Guns/Battery/battery_guns.yml(2 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml(3 hunks)Resources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/meta.json(4 hunks)Resources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/meta.json(1 hunks)
🧰 Additional context used
🪛 GitHub Actions: YAML Linter
Resources/Locale/ru-RU/ss14-ru/prototypes/entities/objects/weapons/guns/battery/battery_guns.ftl
[error] 70-70: Expected one of 'a-zA-Z'.
🪛 GitHub Actions: Build & Test Debug
Resources/Locale/ru-RU/ss14-ru/prototypes/entities/objects/weapons/guns/battery/battery_guns.ftl
[error] 70-70: Expected one of "a-zA-Z"
🪛 GitHub Actions: RSI Validator
Resources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/meta.json
[error] 125-125: Failed to read meta.json: Expecting property name enclosed in double quotes.
🪛 YAMLlint (1.35.1)
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml
[warning] 22-22: wrong indentation: expected 8 but found 10
(indentation)
[error] 44-44: trailing spaces
(trailing-spaces)
[error] 107-107: trailing spaces
(trailing-spaces)
🔇 Additional comments (10)
Resources/Locale/en-US/ss14-ru/prototypes/entities/objects/weapons/guns/battery/battery_guns.ftl (1)
53-54: Отлично! Локализация соответствует описанию оружия.Описание точно отражает характеристики оружия, указанные в целях PR.
Resources/Locale/ru-RU/ss14-ru/prototypes/entities/objects/weapons/guns/battery/battery_guns.ftl (1)
68-69: Хороший перевод!Перевод точно передает смысл оригинала и соответствует стилю игры.
Resources/Textures/Objects/Weapons/Guns/Battery/antiquelasergun.rsi/meta.json (1)
18-25: Анимация корректно настроена!Задержки анимации правильно определены для всех состояний оружия.
Also applies to: 42-59, 64-81
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml (4)
26-30: Замена типа боеприпаса на BulletEnergyGunLaserMk2
В компонентах ProjectileBatteryAmmoProvider и BatteryWeaponFireModes обновлено значение поля proto на "BulletEnergyGunLaserMk2", что соответствует обновленной механике выстрелов. Проверьте, что все ссылки и определения обновлены согласно новым требованиям.
61-88: Добавление нового типа снаряда BulletEnergyGunLaserMk2
Новая сущность с именем "лазерная пуля" и идентификатором BulletEnergyGunLaserMk2 включает компоненты Reflective, FlyBySound, Sprite и Projectile. Обратите внимание, что урон теперь задан как Heat: 20, что соответствует изменению баланса (уменьшение с 22 до 20).
89-107: Внедрение визуального эффекта BulletImpactEffectRedDisabler
Новая сущность BulletImpactEffectRedDisabler добавлена с компонентами TimedDespawn (lifetime: 0.2), Sprite и EffectVisuals. Проверьте, что параметры визуального эффекта соответствуют требованиям игрового баланса.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 107-107: trailing spaces
(trailing-spaces)
18-22: 🛠️ Refactor suggestionОбновление параметров компонента Gun
Здесь изменена скорость снаряда (projectileSpeed: 40) и добавлен новый блок soundGunshot с обновленным путём и параметрами (volume: -2). Обратите внимание, что YAMLlint сообщает о неправильном отступе на строке 22 (ожидается 8 пробелов, найдено 10).🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 22-22: wrong indentation: expected 8 but found 10
(indentation)
Resources/Prototypes/Entities/Objects/Weapons/Guns/Battery/battery_guns.yml (3)
580-584: Обновление параметров компонента Gun в antique laser pistol
В секции компонента Gun для сущности "antique laser pistol" изменена скорость снаряда на 40, а также обновлен звук выстрела: изменён путь к файлу звука на "/Audio/Backmen/Weapons/Energy/Resources_Audio_DeltaV_Weapons_Guns_Gunshots_laser.ogg" и добавлен параметр volume: -2. Эти изменения соответствуют новым требованиям баланса.
586-586: Обновление типа боеприпаса для antique laser pistol
В компоненте ProjectileBatteryAmmoProvider для antique laser pistol изменено значение поля proto на "BulletEnergyGunLaserMk2", что гарантирует использование нового типа снаряда.
602-602: Изменение статической цены
Статическая цена для antique laser pistol обновлена до 5750, что отражает изменения в экономике игры. Убедитесь, что данное значение цены интегрируется с общей системой расчёта стоимости предметов.
...urces/Locale/ru-RU/ss14-ru/prototypes/entities/objects/weapons/guns/battery/battery_guns.ftl
Outdated
Show resolved
Hide resolved
Resources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/meta.json
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/weapons/guns/battery/X01.ftl(1 hunks)Resources/Prototypes/Entities/Objects/Weapons/Guns/Battery/battery_guns.yml(2 hunks)Resources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/meta.json(4 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- Resources/Textures/Backmen/Objects/Weapons/Guns/Energy/x01.rsi/meta.json
🧰 Additional context used
🪛 GitHub Actions: YAML Linter
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/weapons/guns/battery/X01.ftl
[error] 3-3: Expected one of 'a-zA-Z'.
⏰ Context from checks skipped due to timeout of 300000ms (2)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (4)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/weapons/guns/battery/X01.ftl (1)
1-2: Перевод корректен и соответствует функционалу оружия.Перевод точно отражает характеристики оружия и его происхождение.
Resources/Prototypes/Entities/Objects/Weapons/Guns/Battery/battery_guns.yml (3)
580-584: Параметры звука и скорости снаряда соответствуют балансу.Изменения скорости снаряда и настройки звука улучшают баланс оружия.
585-587: Тип снаряда изменен в соответствии с целями PR.Замена типа снаряда на
BulletEnergyGunLaserMk2соответствует указанным изменениям.
602-602: Значительное увеличение цены.Цена увеличена с 750 до 5750, что существенно влияет на баланс игры. Убедитесь, что это намеренное изменение.
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/weapons/guns/battery/X01.ftl
Outdated
Show resolved
Hide resolved
|
Этот PR проклят... |
Pomogite...
|
Проклятие 220... |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml (2)
44-44: Удалите лишние пробелы.
Обнаружены хвостовые пробелы на строке 44. Рекомендуется удалить их для соответствия стандартам оформления YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 44-44: trailing spaces
(trailing-spaces)
89-107: Добавление эффекта при попадании "BulletImpactEffectRedDisabler".
Новый прототип эффекта добавлен с заданными компонентами TimedDespawn, Sprite, EffectVisuals и Tag. Обратите внимание, что в строке 107 имеются лишние пробелы – их необходимо удалить для избежания ошибок форматирования YAML.Следующий diff демонстрирует удаление лишних пробелов:
- +🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 107-107: trailing spaces
(trailing-spaces)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml(3 hunks)
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml
[warning] 22-22: wrong indentation: expected 8 but found 10
(indentation)
[error] 44-44: trailing spaces
(trailing-spaces)
[error] 107-107: trailing spaces
(trailing-spaces)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (4)
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Energy/hitscan.yml (4)
26-31: Обновление ссылки на новый прототип снаряда.
В компоненте ProjectileBatteryAmmoProvider и в списке fireModes компонента BatteryWeaponFireModes изменена ссылка на прототип снаряда с устаревшего (X01Attack) на новый BulletEnergyGunLaserMk2. Это изменение соответствует заявленным балансировочным корректировкам.
45-60: Удаление устаревшего прототипа "X01Attack".
Устаревший тип снаряда (X01Attack) удалён, что соответствует переходу на новый прототип BulletEnergyGunLaserMk2. Убедитесь, что все ссылки на X01Attack обновлены в остальной части кодовой базы.
61-88: Добавление нового прототипа "BulletEnergyGunLaserMk2".
Добавлена новая сущность для лазерной пули, включающая компоненты Reflective, FlyBySound, Sprite и Projectile. Значение урона для типа Heat установлено в 20, что соответствует изменению баланса (снижение с 22 до 20). Все изменения выглядят корректно – проверьте, что настройка звуковых эффектов соответствует требованиям проекта.
18-22:⚠️ Potential issueПроверка параметров "Gun" компонента.
Изменено значение скорости снаряда (projectileSpeed: 40) и добавлен блок soundGunshot с указанием пути к звуковому файлу, что соответствует требованиям PR. Однако обратите внимание, что YAMLlint сигнализирует об ошибке отступа в строке с volume – ожидается 8 пробелов, а найдено 10.Следующий diff предлагает исправление отступа для строки volume:
- volume: -2 + volume: -2🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 22-22: wrong indentation: expected 8 but found 10
(indentation)
Описание PR
Данный PR изменяет текстуры X-01 multiphase gun и антикварного лазерного пистолета. Помимо этого вышеуказанные лазеры теперь стреляют одним типом боевого лазера. Так же был добавлен перевод для X-01 multiphase gun на русский язык.
Медиа
Тип PR
🆑
Summary by CodeRabbit