Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Review] RatKing Tweaks and Changes #2500

Open
wants to merge 35 commits into
base: master
Choose a base branch
from

Conversation

Kiiwwwie
Copy link

@Kiiwwwie Kiiwwwie commented Jan 19, 2025

Описание PR

Некоторые изменения в отношении крысиного короля. Часть идей ещё в процессе (но если у меня не хватит сил я оставлю всё что уже готово, а не готовое вырежу)

Почему были изменены шансы нахождения еды: В данный момент шанс найти еду в куче 25%, а в мусорке 60%. Также сама еда из мусорок лучше. Сделано это в связи с тем, что баланс антагониста в текущий момент не учитывает шанса быть обнаруженным ИИ просто по факту нахождения в коридоре и в целом опасности нахождения вне технических тоннелей.

При этом стоит понимать что чейндж не делает так, что каждая вторая мусорка хранит в себе сырный пирог. Выше стал только шанс нахождения какой-либо еды в принципе. Колёса сыра в целом даже реже чем раньше. Шанс найти сырный пирог такой, что вряд ли будет больше одного такого на смену, если вообще будет.
Планы: (Мои собственные, могут быть изменены)

  • Добавить ККоролю вариацию чуткого слуха таяр
  • Изменять курсор при перекапывании мусора
  • Добавить поп-ап уведомление когда начинаешь раскапывать мусор, пытаешься раскопать уже раскопанный мусор и когда пытаешься раскопать не мусор
  • Перенос спавна ивентовых мышей на вентиляции
  • добавить крысиному королю возможность разделывать трупы животных (Сделано оффами)
  • Сделать так чтобы ивент крысиного короля работал как задумано
  • Добавить Крысиному королю возможность спавниться в мусорных баках
  • Отдельный пулл с лутом в мусорных баках

Медиа

Проверки

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

🆑 kimorue, Kemran, Kirus

  • add: Добавлена кнопка быстрого действия для способности Крысиного Короля раскапывать еду
  • add: Добавлен чуткий слух для Крысиного короля
  • add: Добавлена возможность раскапывать еду в кучах мусора, мусорных баках и коробках будучи Крысиным Королём
  • add: Добавлены новые спрайты для Короны Крысиного Короля, способности призыва Крысиного Слуги и Королевского Домена
  • add: Добавлена возможность таскать вещи хвостом, будучи крысиным королём
  • tweak: В мусорках теперь больше разнообразного сыра!
  • tweak: Теперь Крысиный король появляется из мусорных куч и баков, а не из вентиляции.
  • fix: Коробки для еды теперь возможно разрушить

@github-actions github-actions bot added Changes: Sprites Изменение затронуло файлы ".rsu / .png" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: C# Изменение затронуло файлы ".cs" labels Jan 19, 2025
Copy link

github-actions bot commented Jan 19, 2025

RSI Diff Bot; head commit deef19c merging into 6946cf2
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/SS220/Clothing/Head/Misc/fancycrown.rsi

State Old New Status
equipped-HELMET Added
icon Added

Resources/Textures/SS220/Interface/Actions/actions_rat_king.rsi

State Old New Status
ratKingArmy Added
ratKingDomain Added
rummage Added

Edit: diff updated after deef19c

@stalengd stalengd marked this pull request as draft January 19, 2025 21:58
@UrPrice UrPrice requested review from UrPrice and Ady4ik January 20, 2025 07:13
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added the Merge Conflict Some content has conflicts. label Jan 21, 2025
@github-actions github-actions bot removed the Merge Conflict Some content has conflicts. label Jan 21, 2025
@Kiiwwwie Kiiwwwie changed the title [WiP] RatKing Tweaks and Changes RatKing Tweaks and Changes Jan 22, 2025
@Kiiwwwie Kiiwwwie changed the title RatKing Tweaks and Changes [WiP] RatKing Tweaks and Changes Jan 22, 2025
@Kiiwwwie Kiiwwwie changed the title [WiP] RatKing Tweaks and Changes RatKing Tweaks and Changes Jan 22, 2025
@UrPrice
Copy link
Collaborator

UrPrice commented Jan 24, 2025

Я попробую вернуть всё с объявой, как и должно быть, и перенесу спавн мышек на ивенте обратно на вентиляции.\n\nНе хочу делать отдельную объяву с жизненными формами в техах из-за меты

Оставить на мусорных кучах можешь, только проверь, что они имеются на всех картах в упоминаниях. Объявление менять не нужно, мусорные кучи или венты в объявлении - не так важно.

@UrPrice
Copy link
Collaborator

UrPrice commented Jan 24, 2025

Окей, перехожу обратно в WiP пока не сделаю (пока что я не в курсе как)

👍

@Kiiwwwie
Copy link
Author

Оставить на мусорных кучах можешь, только проверь, что они имеются на всех картах в упоминаниях. Объявление менять не нужно, мусорные кучи или венты в объявлении - не так важно.

Я изначально хотел чтобы мыши спавнились на вентах, а КК на мусорках. Просто не понимал как. Сейчас я смог это сделать. Объяву оставлю, потому что это способ сообщить гостам что есть ролька. И так как на вентах будут мыши, объява не будет себе противоречить.

@UrPrice
Copy link
Collaborator

UrPrice commented Jan 25, 2025

пока что я не в курсе как

Как я помню, у косм. ниндзя есть блинк, который как раз натягивает иконку экшна на курсор.

@Kiiwwwie Kiiwwwie changed the title [WiP] RatKing Tweaks and Changes [Review] RatKing Tweaks and Changes Jan 25, 2025
@Kiiwwwie Kiiwwwie requested a review from UrPrice January 25, 2025 20:06
@Kiiwwwie
Copy link
Author

@UrPrice Review

Content.Shared/RatKing/RatKingComponent.cs Outdated Show resolved Hide resolved
Content.Shared/RatKing/RatKingActions.cs Outdated Show resolved Hide resolved
Content.Shared/RatKing/SharedRatKingSystem.cs Outdated Show resolved Hide resolved
Content.Shared/RatKing/SharedRatKingSystem.cs Outdated Show resolved Hide resolved
Resources/Prototypes/Catalog/Fills/Boxes/general.yml Outdated Show resolved Hide resolved
@Kiiwwwie Kiiwwwie marked this pull request as ready for review January 26, 2025 13:14
@Kiiwwwie Kiiwwwie requested a review from stalengd January 26, 2025 13:15
@Kiiwwwie
Copy link
Author

Что этому вашему убунту не нравится уээ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Изменение затронуло файлы ".cs" Changes: Localization Изменение затронуло файлы ".ftl" Changes: Prototypes Изменение затронуло файлы ".yml" кроме неймспейса "maps" Changes: Sprites Изменение затронуло файлы ".rsu / .png" Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants