Респрайт большинства ID-карт. #993
Conversation
WalkthroughИзменения затрагивают файлы, связанные с идентификационными картами в игре. Были обновлены метаданные и прототипы карт, включая переименование состояний, изменение атрибуции авторства и добавление новых визуальных состояний. Модификации коснулись как файлов текстур, так и описаний сущностей, что указывает на комплексное обновление системы идентификационных карт. Changes
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/XS
S: Untriaged
Changes: Sprites
|
RSI Diff Bot; head commit a8d9c72 merging into f9aa07e Resources/Textures/Backmen/Objects/Misc/id_cards.rsi
Resources/Textures/Objects/Misc/id_cards.rsi
|
![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
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json(3 hunks)
🔇 Additional comments (3)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json (3)
Line range hint 137-281: Проверьте полноту обновления спрайтов
Большое количество добавленных состояний ID-карт требует проверки наличия соответствующих спрайтов и их качества. Убедитесь, что:
- Все новые состояния имеют соответствующие спрайты
- Спрайты соответствуют общему стилю игры
- Нет пропущенных или устаревших состояний
Рекомендуется добавить комментарии к группам состояний для улучшения читаемости файла meta.json. Например:
"states": [
+ // Administrative cards
{
"name": "admin"
},
+ // ERT cards
{
"name": "ert_chaplain"
},✅ Verification successful
Все спрайты ID-карт присутствуют и корректны
Проверка показала, что:
- Все состояния ID-карт, указанные в meta.json, имеют соответствующие .png файлы
- Не обнаружено подозрительно маленьких или пустых спрайтов (менее 100 байт)
- Все спрайты находятся в ожидаемой директории Resources/Textures/Objects/Misc/id_cards.rsi/
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия спрайтов для всех состояний
jq -r '.states[].name' Resources/Textures/Objects/Misc/id_cards.rsi/meta.json | while read state; do
if [[ ! $state =~ "inhand" ]]; then
echo "Проверка спрайта для состояния: $state"
test -f "Resources/Textures/Objects/Misc/id_cards.rsi/$state.png"
fi
done
Length of output: 16769
Script:
#!/bin/bash
# Проверяем наличие файлов .png для всех состояний и выводим отсутствующие
echo "Отсутствующие спрайты:"
jq -r '.states[].name' Resources/Textures/Objects/Misc/id_cards.rsi/meta.json | while read state; do
if [[ ! $state =~ "inhand" ]] && [ ! -f "Resources/Textures/Objects/Misc/id_cards.rsi/$state.png" ]; then
echo "- $state"
fi
done
# Проверяем размеры файлов, чтобы убедиться что они не пустые
echo -e "\nПодозрительно маленькие спрайты (менее 100 байт):"
find Resources/Textures/Objects/Misc/id_cards.rsi -name "*.png" -size -100c -exec ls -lh {} \;
Length of output: 10249
32-41: Проверьте согласованность переименований
Переименование состояний ID-карт может повлиять на другие части кодовой базы. Необходимо убедиться, что все ссылки на старые названия обновлены:
gold→idcentcomidpassenger→idgeneticistidatmospherictechnician→idlawyer
4-4: Проверьте корректность информации об авторских правах
Изменение атрибуции авторских прав с нескольких авторов на одного требует подтверждения. Убедитесь, что:
- Все спрайты действительно были переработаны
- Предыдущие авторы согласны с удалением их упоминаний
- Лицензия CC-BY-SA-3.0 соблюдается корректно
| "name": "gold-inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idbrigmedic" | ||
| "name": "gold-inhand-right", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idjanitor" | ||
| "name": "default-inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idlawyer" | ||
| "name": "default-inhand-right", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idiaa" | ||
| "name": "silver-inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idmedicaldoctor" | ||
| "name": "silver-inhand-right", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idmime" | ||
| "name": "orange-inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idparamedic" | ||
| "name": "orange-inhand-right", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idpsychologist" | ||
| "name": "blue-inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idreporter" | ||
| "name": "blue-inhand-right", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idprisoner" | ||
| "name": "green-inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "idquartermaster" | ||
| "name": "green-inhand-right", | ||
| "directions": 4 |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружено несоответствие между метаданными и файлами спрайтов
В meta.json указано "directions": 4 для спрайтов в руках, что подразумевает наличие 4 направлений (0-3) для каждого спрайта. Однако фактически присутствуют только базовые спрайты без нумерации направлений (например, blue-inhand-left.png вместо ожидаемых blue-inhand-left.0.png, blue-inhand-left.1.png и т.д.).
Необходимо:
- Либо создать спрайты для всех 4 направлений
- Либо изменить значение
directionsна 1 в meta.json, если анимация для разных направлений не требуется
🔗 Analysis chain
Проверьте корректность направлений спрайтов
Добавление атрибута directions: 4 для спрайтов в руках - правильное решение. Однако нужно убедиться, что все необходимые спрайты присутствуют в текстурах.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия всех необходимых спрайтов
for direction in {0..3}; do
for hand in left right; do
for type in gold default silver orange blue green; do
echo "Проверка: $type-inhand-$hand.$direction.png"
test -f "Resources/Textures/Objects/Misc/id_cards.rsi/$type-inhand-$hand.$direction.png"
done
done
done
Length of output: 10407
Script:
#!/bin/bash
# Проверяем наличие файлов спрайтов с помощью ls, чтобы увидеть, какие файлы существуют
ls -la Resources/Textures/Objects/Misc/id_cards.rsi/*-inhand-*.png
Length of output: 2224
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json (1)
Line range hint
137-281: Рекомендации по улучшению структуры файлаДля улучшения читаемости и поддерживаемости файла предлагается:
- Сгруппировать состояния по категориям (базовые карты, карты отделов, специальные карты)
- Добавить комментарии или описания для каждой группы состояний
Пример структуры:
{ "states": [ // Базовые карты { "name": "default" }, { "name": "silver" }, { "name": "gold" }, // Карты отделов { "name": "idchiefengineer" }, { "name": "idcaptain" }, // Специальные карты { "name": "syndie" }, { "name": "prisoner_001" } ] }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json(3 hunks)
🔇 Additional comments (2)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json (2)
89-134: Корректная структура направленных спрайтов
Добавленные состояния для спрайтов в руках имеют правильную структуру:
- Каждый вариант имеет левую и правую версию
- Все состояния корректно определяют 4 направления
32-41: Проверить соответствие переименований игровой логике
Замечены существенные изменения в названиях состояний ID-карт. Необходимо подтвердить, что:
- Новые названия соответствуют актуальным ролям в игре
- Переименования не нарушат работу существующего кода
Например:
- idcentcom (строка 32)
- idgeneticist (строка 35)
- idlawyer (строка 38)
✅ Verification successful
Let me generate additional verification scripts to check the new state names and their usage.
Изменения в состояниях ID-карт соответствуют игровой логике
На основе проверки кодовой базы подтверждаю, что:
- Новое состояние
idcentcomактивно используется в прототипах для карт Центрального Командования - Состояние
idgeneticistправильно определено для карт генетиков - Состояние
idlawyerиспользуется в прототипах для карт юристов - Все новые состояния корректно определены в meta.json файле спрайтов
Изменения в именах состояний согласуются с существующими конвенциями именования в кодовой базе и отражают соответствующие роли персонала.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка использования старых названий состояний в коде
rg "gold|idpassenger|idatmospherictechnician" --type yaml
Length of output: 26670
Script:
#!/bin/bash
# Check for references to the new ID card states
rg "idcentcom|idgeneticist|idlawyer|idunknown" --type yaml
# Check the full content of the meta.json file
cat Resources/Textures/Objects/Misc/id_cards.rsi/meta.json
Length of output: 5312
| @@ -1,18 +1,12 @@ | |||
| { | |||
| "version": 1, | |||
| "license": "CC-BY-SA-3.0", | |||
| "copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/d917f4c2a088419d5c3aec7656b7ff8cebd1822e idcluwne made by brainfood1183 (github) for ss14, idbrigmedic made by PuroSlavKing (Github), pirate made by brainfood1183 (github), idadmin made by Arimah (github), idvisitor by IProduceWidgets (Github), idintern-service by spanky-spanky (Github)", | |||
| "copyright": "Resprited ALL by CatBackGround(GitHub).", | |||
There was a problem hiding this comment.
💡 Codebase verification
Необходимо сохранить информацию о лицензии и авторских правах
Текущее изменение некорректно с точки зрения лицензии CC-BY-SA-3.0, которая требует указания авторства оригинальных работ. Даже при полном респрайте необходимо сохранять информацию об оригинальных авторах, так как новые спрайты являются производной работой.
Предлагаемые исправления:
- "copyright": "Resprited ALL by CatBackGround(GitHub).",
+ "copyright": "Оригинальные спрайты от Space Wizards и tgstation под CC-BY-SA-3.0. Полный респрайт выполнен CatBackGround(GitHub)."• Файл уже содержит правильную лицензию ("license": "CC-BY-SA-3.0"), но требуется корректное указание авторства в соответствии с условиями этой лицензии.
🔗 Analysis chain
Необходимо сохранить историю авторства
Текущее изменение удаляет информацию об оригинальных авторах спрайтов. Даже при полном респрайте, рекомендуется сохранить ссылку на оригинальные работы, от которых произошли изменения.
Предлагаемое исправление:
- "copyright": "Resprited ALL by CatBackGround(GitHub).",
+ "copyright": "Original sprites from tgstation and SS14 contributors, resprited by CatBackGround(GitHub).",Let me verify the copyright attribution history by checking the git history and related files.
Let me check the content of these license files and README to understand the project's attribution requirements.
Let me fix the command to properly search for copyright information in RSI files.
Let me check the specific RSI file content to understand the previous copyright information.
🏁 Scripts executed
The following scripts were executed for the analysis:
Error: No shell scripts found
Script:
#!/bin/bash
# Check git history for the specific file
git log -p -- Resources/Textures/Objects/Misc/id_cards.rsi/meta.json | head -n 50
# Look for any README or license files in the texture directory
fd -t f "README|LICENSE" Resources/Textures/
# Check for any attribution or copyright files in the root
fd -t f -d 1 "^(COPYRIGHT|LICENSE|CREDITS|README)"
Length of output: 144
Script:
#!/bin/bash
# Check README.md for attribution guidelines
cat README.md | grep -A 5 -i "attribution\|credit\|copyright"
# Check content of both license files
head -n 10 LICENSE-AGPLv3.txt
head -n 10 LICENSE-MIT.TXT
# Let's also check if there are any other RSI files with copyright information
fd -e json -x grep -l "copyright" {} \; Resources/Textures/
Length of output: 1448
Script:
#!/bin/bash
# Search for copyright information in RSI metadata files
cd Resources/Textures && fd -e json -x grep -l "copyright" {}
# Also check if there are any specific guidelines for sprites
cd ../../
fd "SPRITING|ARTWORK|ART" -i
Length of output: 65641
Script:
#!/bin/bash
# Check the content of the ID cards meta.json file
cat Resources/Textures/Objects/Misc/id_cards.rsi/meta.json
# Check if there are any contribution guidelines
cat README.md | head -n 30
Length of output: 6034
|
мне не нравится, удаляй |
Терпи. |
|
если ты собрался этой хуйнёй маяться, то почему бы не сделать нормально? |
|
|
|
|
There was a problem hiding this comment.
Actionable comments posted: 2
♻️ Duplicate comments (1)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json (1)
4-4:⚠️ Potential issueНеобходимо сохранить информацию об оригинальных авторах
Текущее изменение нарушает условия лицензии CC-BY-SA-3.0, которая требует сохранения информации об оригинальных авторах даже при полном респрайте, так как новые спрайты являются производной работой.
Предлагаемое исправление:
- "copyright": "Resprited ALL by CatBackGround(GitHub) & fizzoghoster(Discord).", + "copyright": "Оригинальные спрайты от Space Wizards и tgstation под CC-BY-SA-3.0. Респрайт выполнен CatBackGround(GitHub) и fizzoghoster(Discord)."
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json(3 hunks)
🔇 Additional comments (1)
Resources/Textures/Objects/Misc/id_cards.rsi/meta.json (1)
89-134: Проверьте наличие спрайтов для всех направлений
В мета-данных указано "directions": 4 для спрайтов в руках, что требует наличия спрайтов для всех 4 направлений. Убедитесь, что все необходимые спрайты присутствуют в текстурах.
#!/bin/bash
# Проверка наличия всех необходимых спрайтов для каждого направления
for type in gold default silver orange blue green; do
for hand in left right; do
for dir in {0..3}; do
echo "Проверка: $type-inhand-$hand.$dir.png"
test -f "Resources/Textures/Objects/Misc/id_cards.rsi/$type-inhand-$hand.$dir.png"
done
done
done| "name": "iddetective" | ||
| }, | ||
| { | ||
| "name": "idheadofpersonnel" | ||
| }, | ||
| { | ||
| "name": "idiaa" |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружены проблемы с реализацией новых ID-карт
- Спрайты
idheadofpersonnel.pngиidiaa.pngотсутствуют в RSI директории, хотя состояния определены в meta.json - Прототипы для карт существуют:
idheadofpersonnelвResources/Prototypes/Entities/Objects/Misc/identification_cards.ymlidiaaвResources/Prototypes/_Backmen/Roles/Jobs/CentCom/centcom_official.yml,centcom_admiral.ymlиResources/Prototypes/Corvax/Entities/Objects/Misc/identification_cards.yml
- Локализация для "Head of Personnel" присутствует в обоих языках (en-US и ru-RU), но отсутствует полная локализация для "Internal Affairs Agent"
Необходимо:
- Добавить отсутствующие спрайты для состояний
idheadofpersonnelиidiaa - Добавить недостающие строки локализации для должности Internal Affairs Agent
🔗 Analysis chain
Проверьте новые состояния
Добавлены новые состояния:
idheadofpersonnelidiaa
Убедитесь, что:
- Спрайты для этих состояний присутствуют в RSI
- Прототипы для новых ID-карт созданы
- Локализация для новых должностей добавлена
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия необходимых файлов и записей
echo "Проверка спрайтов..."
test -f "Resources/Textures/Objects/Misc/id_cards.rsi/idheadofpersonnel.png"
test -f "Resources/Textures/Objects/Misc/id_cards.rsi/idiaa.png"
echo "Проверка прототипов..."
rg "idheadofpersonnel|idiaa" Resources/Prototypes/
echo "Проверка локализации..."
rg "head-of-personnel|internal-affairs-agent" Resources/Locale/
Length of output: 1127
| "name": "idcentcom" | ||
| }, | ||
| { | ||
| "name": "idpassenger" | ||
| "name": "idgeneticist" | ||
| }, | ||
| { | ||
| "name": "idatmospherictechnician" | ||
| "name": "idlawyer" | ||
| }, | ||
| { | ||
| "name": "idbartender" | ||
| "name": "idunknown" |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружены несоответствия в переименованиях ID-карт
В файле Resources/Prototypes/Entities/Objects/Misc/identification_cards.yml найдены следующие несоответствия:
- Состояние
goldвсё ещё используется в нескольких ID-картах (IDSFO, ERTLeaderIDCard) - Состояние
idpassengerиспользуется в AgentIDCard - Состояние
idatmospherictechnicianиспользуется в AtmosIDCard
Необходимо обновить следующие прототипы для соответствия новым именам состояний:
- IDSFO (заменить
goldна новое состояние) - ERTLeaderIDCard (заменить
goldна новое состояние) - AgentIDCard (обновить
idpassengerнаidgeneticist) - AtmosIDCard (обновить
idatmospherictechnicianнаidlawyer)
🔗 Analysis chain
Проверьте согласованность переименований
Произведены значительные изменения в именах состояний:
gold→idcentcomidpassenger→idgeneticistidatmospherictechnician→idlawyer- Добавлено
idunknown
Необходимо убедиться, что все сущности и прототипы, использующие эти состояния, также обновлены.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск использования старых имен состояний
echo "Поиск использования старых имен..."
rg -l "gold|idpassenger|idatmospherictechnician" Resources/Prototypes/
echo "Поиск использования новых имен..."
rg -l "idcentcom|idgeneticist|idlawyer|idunknown" Resources/Prototypes/
Length of output: 5253
Script:
#!/bin/bash
# Проверим конкретные файлы с ID-картами на предмет несоответствий
echo "Содержимое файла _Backmen/Entities/Objects/Misc/identification_cards.yml:"
cat "Resources/Prototypes/_Backmen/Entities/Objects/Misc/identification_cards.yml"
echo -e "\nСодержимое файла Entities/Objects/Misc/identification_cards.yml:"
cat "Resources/Prototypes/Entities/Objects/Misc/identification_cards.yml"
Length of output: 28675
А ты инвалид и не можешь нормально сделать? |
Моё дело - респрайт, я не буду трогать ебучие прототипы. |
Описание PR
Текущие огрызки, называемые ID-картами, требовали косметического респрайта. Получите, и распишитесь.
Медиа
Soon...
Тип PR
Изменения
🆑 CatBackGround
Summary by CodeRabbit
Визуальные изменения
Обновления