Another announcer #372
Another announcer #372
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the WalkthroughИзменения включают добавление нового диктора A.C.E.S в игру "Lost Paradise". Обновления затрагивают файлы локализации на английском и русском языках, конфигурацию дикторов и звуковые коллекции. Вес дикторов был перераспределен между тремя персонажами: NTH, AFONYA и новым ACES, каждый с равным весом 0.33. Добавлены новые переводы и описания для различных игровых элементов. Changes
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
Статус: Требуется одобрение
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (47)
Resources/Audio/Announcers/_ACES/alerts/blue.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/delta.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/epsilon.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/gamma.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/green.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/red.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/violet.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/white.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/alerts/yellow.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/announce.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/attention.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/command_report.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/malf.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/ninja_hacking.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/noospheric_storm.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/powersink_explosion.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/unknown_shuttle.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/comms/war.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/anomaly.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/bluespace_artifact.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/breaker_flip.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/bureaucratic_error.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/cargo-gifts-event-announcement.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/carp_rift.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/clerical_error.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/gas_leak-complete.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/gas_leak.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/immovable_rod_spawn.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/ion_storm.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/kudzu_growth.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/meteors-complete.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/meteors.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/migration.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/mouse_migration.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/noospheric_storm.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/power_grid_check-complete.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/power_grid_check.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/random_sentience.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/revenant_spawn.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/solar_flare-complete.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/solar_flare.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/events/vevent_clog.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/fallback.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/shuttle/called.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/shuttle/docked.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/shuttle/left.oggis excluded by!**/*.oggResources/Audio/Announcers/_ACES/shuttle/recalled.oggis excluded by!**/*.ogg
📒 Files selected for processing (5)
Resources/Locale/en-US/_LostParadise/update20.ftl(1 hunks)Resources/Locale/ru-RU/_LostParadise/update20.ftl(1 hunks)Resources/Prototypes/Announcers/!randomAnnouncers.yml(1 hunks)Resources/Prototypes/_LostParadise/Announcers/ACES.yml(1 hunks)Resources/Prototypes/_LostParadise/SoundCollections/Announcers.yml(1 hunks)
🧰 Additional context used
📓 Path-based instructions (5)
Resources/Locale/ru-RU/_LostParadise/update20.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/_LostParadise/SoundCollections/Announcers.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Announcers/!randomAnnouncers.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Announcers/ACES.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Locale/en-US/_LostParadise/update20.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
🪛 yamllint (1.35.1)
Resources/Prototypes/_LostParadise/SoundCollections/Announcers.yml
[error] 11-11: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (8)
Resources/Locale/ru-RU/_LostParadise/update20.ftl (1)
9-9: Добавление нового диктора выглядит корректно.Перевод соответствует общему стилю и не нарушает правила оформления.
Resources/Locale/en-US/_LostParadise/update20.ftl (1)
9-9: Обратите внимание на языковой контекст.Данный файл помечен как en-US, но содержит текст на русском языке. Проверьте, действительно ли требуется смешение языков в рамках одной локализации и соответствует ли это вашим требованиям.
Resources/Prototypes/Announcers/!randomAnnouncers.yml (1)
4-6: Уточните суммарный вес дикторов.Сумма весов (0.33 + 0.33 + 0.33) равна 0.99, а не 1. Уточните, не нужно ли округлить одно из значений, чтобы в сумме получить 1.0, если движок вашей системы ожидает точной суммы.
Resources/Prototypes/_LostParadise/SoundCollections/Announcers.yml (1)
6-11: 🧹 Nitpick (assertive)Добавьте перевод строки в конец файла.
Статический анализ указывает на отсутствие перевода строки в конце файла. Рекомендуется добавить пустую строку, чтобы устранить предупреждение.
... - /Audio/Announcers/_ACES/comms/announce.ogg +Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 11-11: no new line character at the end of file
(new-line-at-end-of-file)
Resources/Prototypes/_LostParadise/Announcers/ACES.yml (4)
28-45: Корректная конфигурация уровней тревогиВсе уровни тревоги определены правильно, с понятными описаниями и последовательным именованием файлов.
113-130: Базовые оповещения шаттла реализованы корректноОсновные оповещения шаттла (вызов, отзыв, стыковка, отбытие) реализованы правильно. Закомментированные секции могут быть использованы для будущего расширения функциональности.
132-134: Корректная конфигурация резервного оповещенияРезервное оповещение настроено правильно и будет использоваться для всех неопределённых типов объявлений.
24-25: Проверьте существование коллекции ACESWelcomeУбедитесь, что коллекция звуков
ACESWelcomeопределена в соответствующем файле конфигурации.✅ Verification successful
Коллекция звуков ACESWelcome корректно определена
Коллекция
ACESWelcomeнайдена в файлеResources/Prototypes/_LostParadise/SoundCollections/Announcers.ymlи содержит два звуковых файла:
/Audio/Announcers/_ACES/comms/malf.ogg/Audio/Announcers/_ACES/comms/announce.ogg🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Поиск определения коллекции ACESWelcome rg "ACESWelcome" Resources/Prototypes/Length of output: 208
Script:
#!/bin/bash # Проверим содержимое файла с определением коллекции ACESWelcome cat Resources/Prototypes/_LostParadise/SoundCollections/Announcers.ymlLength of output: 365
| # - id: spawnAnnounce # Captain join # TODO That system is annoyingly not modular | ||
| # path: comms/spawn_announce.ogg |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Требуется доработка системы spawnAnnounce
TODO комментарий указывает на проблему с модульностью системы. Рекомендуется создать задачу для улучшения модульности этой функциональности.
Хотите, чтобы я создал GitHub Issue для отслеживания этой задачи?
| path: events/mouse_migration.ogg | ||
| - id: cockroachMigration # Several cockroaches have appeared in a random place | ||
| path: events/migration.ogg |
There was a problem hiding this comment.
🧹 Nitpick (assertive)
Рассмотрите возможность уникальных звуков для разных событий
Несколько разных событий используют один и тот же файл migration.ogg:
- Миграция мышей
- Миграция тараканов
- Появление слизней
- Появление пауков
Рекомендуется создать уникальные звуки для каждого типа события для лучшего различения.
Also applies to: 97-100
| id: ACES # Localized as "announcer-<id>-name" in chat | ||
| basePath: /Audio/Announcers/_ACES # Where to start looking for audio files | ||
| baseAudioParams: # Default audio parameters for all announcements, all options explained in the template announcement | ||
| volume: -1 # If this announcer is really loud, lower it to match the others' volume #? Default is 3 |
There was a problem hiding this comment.
💡 Codebase verification
Громкость диктора требует корректировки
Анализ других конфигураций дикторов показывает, что значение -1 действительно нестандартное. Типичные значения громкости:
- Шаблон (template): -7
- Michael: -3
- Neil: -3
- Intern: -7
Рекомендуется установить громкость в диапазоне от -7 до -3, чтобы соответствовать существующим дикторам. Значение -1 может сделать диктора слишком громким по сравнению с остальными.
🔗 Analysis chain
Проверьте настройку громкости
Значение громкости -1 кажется нестандартным. Рекомендуется:
- Сравнить с громкостью других дикторов
- Уточнить значение по умолчанию (в комментарии указано неуверенное значение)
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск настроек громкости в других конфигурациях дикторов
rg "volume:" Resources/Prototypes/Announcers/
Length of output: 533
|
Йоу, ахуй |
|
В этом ПР'е есть конфликты, пожалуйста, устраните их, прежде чем мы сможем оценить ваш ПР. |
Описание обновления:
добавление ещё одного аннаунсера A.C.E.S
всё аудио было сделано BasiaBelov (за что ему огромное спасибо)
Медиа
2025-01-03.15-38-52.mp4
Проверки
Изменения
🆑 Kest