[Fix] dop_foxes_sound #1053
[Fix] dop_foxes_sound #1053
Conversation
|
""" WalkthroughВ этом pull request внесены изменения в несколько файлов. В файле Changes
Sequence Diagram(s)sequenceDiagram
participant Пользователь
participant Приложение
participant КонфигурацияЭмотов
Пользователь->>Приложение: Запрос эмота "Howl" или "Growl"
Приложение->>КонфигурацияЭмотов: Проверка доступности
КонфигурацияЭмотов-->>Приложение: Эмот доступен
Приложение->>Пользователь: Проигрывание звука эмота
sequenceDiagram
participant Пользователь
participant Приложение
participant МенеджерЗвуков
Пользователь->>Приложение: Запрос воспроизведения лисьего звука
Приложение->>МенеджерЗвуков: Получение коллекции звуков
МенеджерЗвуков-->>Приложение: Передача списка аудиофайлов (FoxesScream, FoxesCry, ...)
Приложение->>Пользователь: Проигрывание выбранного аудио
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/S
S: Untriaged
No C#
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
Resources/Prototypes/_Backmen/SoundCollections/foxes.yml (1)
15-19: Рекомендация по улучшению разнообразия звуковКоллекции
FoxesGrowlиFoxesHowlсодержат только по одному звуковому файлу. Рекомендуется добавить больше вариаций для этих важных эмоций, чтобы избежать повторяемости звуков.Also applies to: 31-34
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
Resources/Prototypes/Corvax/Voice/speech_emotes.yml(0 hunks)Resources/Prototypes/_Backmen/SoundCollections/foxes.yml(1 hunks)
💤 Files with no reviewable changes (1)
- Resources/Prototypes/Corvax/Voice/speech_emotes.yml
🔇 Additional comments (1)
Resources/Prototypes/_Backmen/SoundCollections/foxes.yml (1)
1-7: Коллекция FoxesScream реализована корректно!Структура YAML и пути к файлам соответствуют стандартам проекта.
| - type: soundCollection | ||
| id: FoxesCry | ||
| files: | ||
| - /Audio/Backmen/Voice/Foxes/foxcry01.ogg | ||
| - /Audio/Backmen/Voice/Foxes/foxcry02.ogg | ||
|
|
||
| - type: soundCollection | ||
| id: FoxesGrowl | ||
| files: | ||
| - /Audio/Backmen/Voice/Foxes/foxgrowl01.ogg | ||
|
|
||
| - type: soundCollection | ||
| id: FoxesPurrs | ||
| files: | ||
| - /Audio/Backmen/Voice/Foxes/foxpurrs01.ogg | ||
| - /Audio/Backmen/Voice/Foxes/foxpurrs02.ogg | ||
|
|
||
| - type: soundCollection | ||
| id: FoxesLaugh | ||
| files: | ||
| - /Audio/Backmen/Voice/Foxes/foxlaugh01.ogg | ||
|
|
||
| - type: soundCollection | ||
| id: FoxesHowl | ||
| files: | ||
| - /Audio/Backmen/Voice/Foxes/foxhowl01.ogg |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Проверьте наличие звуковых файлов
Убедитесь, что все указанные аудиофайлы присутствуют в соответствующих директориях.
🏁 Script executed:
#!/bin/bash
# Проверка наличия звуковых файлов
for file in $(grep -o '/Audio/Backmen/Voice/Foxes/.*\.ogg' Resources/Prototypes/_Backmen/SoundCollections/foxes.yml); do
if [ -f "$file" ]; then
echo "✓ Файл существует: $file"
else
echo "✗ Файл отсутствует: $file"
fi
doneLength of output: 3478
Доработать: отсутствуют указанные звуковые файлы и коллекции с одним файлом нуждаются в улучшении
-
В файле Resources/Prototypes/_Backmen/SoundCollections/foxes.yml обнаружены ссылки на аудиофайлы, которые отсутствуют в репозитории:
/Audio/Backmen/Voice/Foxes/foxscream01.ogg/Audio/Backmen/Voice/Foxes/foxscream02.ogg/Audio/Backmen/Voice/Foxes/foxscream03.ogg/Audio/Backmen/Voice/Foxes/foxscream04.ogg/Audio/Backmen/Voice/Foxes/foxcry01.ogg/Audio/Backmen/Voice/Foxes/foxcry02.ogg/Audio/Backmen/Voice/Foxes/foxgrowl01.ogg/Audio/Backmen/Voice/Foxes/foxpurrs01.ogg/Audio/Backmen/Voice/Foxes/foxpurrs02.ogg/Audio/Backmen/Voice/Foxes/foxlaugh01.ogg/Audio/Backmen/Voice/Foxes/foxhowl01.ogg
-
Рекомендуется:
- Обновить или исправить пути к аудиофайлам, чтобы они соответствовали фактическому расположению файлов.
- Рассмотреть возможность добавления дополнительных звуков для коллекций, где задан только один аудиофайл (например, для коллекций FoxesGrowl, FoxesLaugh, FoxesHowl), если это соответствует целям проекта.
github-actions
bot
added
size/M
Changes: Sprites
|
RSI Diff Bot; head commit 59759b2 merging into 19f835a Resources/Textures/Backmen/Mobs/Customization/fox_markings.rsi
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Resources/Prototypes/_Backmen/Diseases/furry_virus.yml (1)
19-23: Добавлена болезнь FurryVirusPlus.
Новая болезньFurryVirusPlus, наследуемая отFurryVirus, определена без методов лечения (пустой списокcures: []). Убедитесь, что такое поведение (отсутствие лечения) является осознанным решением в рамках баланса игры и корректно обрабатывается игровым механизмом.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (11)
Resources/Audio/Backmen/Voice/Foxes/foxcry01.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxcry02.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxgrowl01.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxhowl01.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxlaugh01.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxpurrs01.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxpurrs02.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxscream01.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxscream02.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxscream03.oggis excluded by!**/*.oggResources/Audio/Backmen/Voice/Foxes/foxscream04.oggis excluded by!**/*.ogg
📒 Files selected for processing (5)
Resources/Audio/Backmen/Voice/Foxes/attributions.yml(1 hunks)Resources/Prototypes/_Backmen/Diseases/furry_virus.yml(1 hunks)Resources/Prototypes/_Backmen/Entities/Mobs/Species/foxes.yml(1 hunks)Resources/Prototypes/_Backmen/Voice/speech_emote_sounds.yml(1 hunks)Resources/Prototypes/_Backmen/Voice/speech_emotes.yml(0 hunks)
💤 Files with no reviewable changes (1)
- Resources/Prototypes/_Backmen/Voice/speech_emotes.yml
✅ Files skipped from review due to trivial changes (1)
- Resources/Audio/Backmen/Voice/Foxes/attributions.yml
🔇 Additional comments (5)
Resources/Prototypes/_Backmen/Entities/Mobs/Species/foxes.yml (1)
70-74: Новый компонент Vocal добавлен корректно.
В данном блоке для сущности Urist McFox добавлен компонентVocalс указанием звуковых коллекций для половых категорий:Male: MaleFoxes,Female: FemaleFoxesиUnsexed: MaleFoxes. Убедитесь, что назначение коллекции для Unsexed соответствует задумке игрового дизайна (то есть, если для небинарных персонажей требуется специфичная коллекция, возможно, стоит использовать другую настройку).Resources/Prototypes/_Backmen/Diseases/furry_virus.yml (2)
11-18: Определение нового эмота BCatMeow.
Новый эмот с идентификаторомBCatMeowдобавлен с категориейVocalи привязан к коллекции звуковFoxesGrowl. Проверьте, что название, локализация (поляnameиchatMessages) и привязка коллекции соответствуют ожиданиям и требованиям игрового процесса.
24-120: Интеграция эмота в эффекты болезни FurryVirus.
В определении болезниFurryVirusэффекты типаDiseaseSnoughтеперь используют эмотBCatMeow. Это изменение должно обеспечить корректное воспроизведение новых звуковых эффектов, связанных с данным заболеванием. Рекомендуется проверить, что YAML-структура (отступы и вложенность) соответствует стандарту, чтобы избежать ошибок при загрузке файла.Resources/Prototypes/_Backmen/Voice/speech_emote_sounds.yml (2)
111-146: Новый набор звуков для MaleFoxes.
Добавлен блокemoteSoundsдля идентификатораMaleFoxes, в котором определены звуковые эффекты для лисиц-мужчин (Scream, Laugh, Growl, Howl, Sneeze, Cough, Yawn, Snore, Sigh, Crying, Whistle, Weh, Gasp, DefaultDeathgasp, Purrs). Проверьте, что все ссылки на коллекции (например,FoxesScream,FoxesLaughи т.д.) соответствуют аудиофайлам, указанным в файлеSoundCollections/foxes.yml, и что параметрvariation: 0.125подходит для выбранного варианта звучания.
148-183: Новый набор звуков для FemaleFoxes.
Добавлен блокemoteSoundsдля идентификатораFemaleFoxes, в котором настроены звуковые эффекты для лисиц-женщин (Scream, Laugh, Growl, Howl, Sneeze, Cough, Yawn, Snore, Sigh, Crying, Whistle, Weh, Gasp, DefaultDeathgasp, Purrs). Убедитесь, что используемые коллекции для женских вариантов, такие какFemaleSneezes,FemaleCoughs,FemaleYawnиFemaleSigh, определены в соответствующих файлах и корректно интегрированы в общий звуковой движок игры.
Описание PR
Исправлено невозможность использовать эмоуты рычанья и завывания + ещё звук мурчанья для лис
Тип PR
Изменения
fix: Исправлена невозможность рычать и завывать на лисах и фелинидах.
add: Добавлен ещё 1 звук мурчанья для лис
Summary by CodeRabbit