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

[UPDATE Xeno and litle FIX] #912

Merged
merged 30 commits into from
Jan 9, 2025
Merged

Conversation

NameLunar
Copy link
Contributor

@NameLunar NameLunar commented Dec 26, 2024

Описание PR

Обновление Ксеноморфов и мелкий фикс священника.

Техническая информация

Если прочитать код, то вроде все понятно.
В компоненте XenoQueen (Серверный) можно изменять некоторые параметры и отключать то или иное.
Очки королевы не показываются и только в компоненте можно посмотреть их количество.
Очки королевы показываются как иконка ниндзя костюма. (Типа цистерна крови. Не обязуйте). Максимум 140. Каждая полоска = 20 очкам.
Причиной этого является отсутствие спрайтов и идей другой кастомизации. Из плюсов можно выделить то, что недостающие очки пишутся при попытке вызвать кого либо.

Медиа

image

Требования

  • Я прочитал(а) и следую Руководство по созданию пулл реквестов. Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению мейнтейнера.
  • Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие его изменения в игре, или этот пулл реквест не требует демонстрации в игре

Чейнджлог
🆑 NameLunar, при поддержке FaDeOkbo

  • add: Ксеноморфы вновь эволюционируют! Их чешуйчатая броня крепка как сталь, но очень уязвима к теплу.
  • fix: Потерянный священник вновь верует в богов!
  • tweak: Теперь Королеве ксеноморфов требуются очки для спавна других ксенов. (Не получится призвать 999 мобов сразу😢)
  • tweak: Штурмовой борг синдиката стал настоящим смертоносным оружием, как и цена на него.

@github-actions github-actions bot added size/L Changes: Localization Изменение локализации S: Untriaged labels Dec 26, 2024
Copy link
Contributor

@Unlumy Unlumy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В целом, практически всё, что я здесь увидел сделано правильно, так что можно будет мерджить если ты поправишь.
Однако .cs файлы бы глянуть кому-нибудь, например @FaDeOkno или @KashRas2

@Unlumy
Copy link
Contributor

Unlumy commented Dec 26, 2024

А, я не увидел, что драфт, но ты всё равно поправь.

@NameLunar
Copy link
Contributor Author

Я не понимаю. Никак не могу понять причину ошибки
[ERRO] proto: Reading Robust.Shared.Prototypes.EntityPrototype(ActionSpawnMobXenoBurrower) threw the following exception: System.Invalid Operation Exception: Type 'Content.Shared.Actions.WorldTargetActionEvent' is abstract, but could not find concrete type 'SpawnXenoQueenEvent'. at Robust.Shared.Serialization.Manager.SerializationManager.ResolveConcreteType(Type baseType, String typeName)

@COTE-LAPYX
Copy link

ПР конечно в драфте, но такие проблемы стоит убирать сразу, а лучше вообще выкинуть привычку все писать в одном пре.

Баг фикс со спавном священника не должен быть тут. Вы же даже ставите галочку что прочитали и следуете руководству по созданию ПРов...
Screenshot_2025-01-03-09-42-15-311_com.android.chrome-edit.jpg

Спасибо.

На счёт ошибки я не уверен до конца не чекал, но пересмотри где регестрируются ивенты на призыв, и сами ивенты.
Ещё скажи, работает ли сам спавн и запускается ли все вообще, либо же это просто проверка шалит.

@github-actions github-actions bot added the Changes: Map Изменение карт label Jan 3, 2025
@NameLunar
Copy link
Contributor Author

Да я уже все исправил. Ошибка была глупая... Про священника да. Но блин, почему и нет? Всего то 10 символов от силы

@NameLunar NameLunar changed the title [X] [UPDATE Xeno and litle FIX Chaplain] Jan 3, 2025
@NameLunar
Copy link
Contributor Author

Сейчас это можно использовать и играть, но проблема точным количеством очков...

@NameLunar NameLunar marked this pull request as ready for review January 3, 2025 14:32
@NameLunar NameLunar closed this Jan 3, 2025
@NameLunar NameLunar reopened this Jan 3, 2025
@NameLunar NameLunar closed this Jan 3, 2025
@NameLunar NameLunar reopened this Jan 3, 2025
@NameLunar NameLunar closed this Jan 3, 2025
@NameLunar NameLunar reopened this Jan 3, 2025
@NameLunar
Copy link
Contributor Author

Ну... Ладно. Займусь позже

@github-actions github-actions bot added the Changes: Sprites Изменение спрайтов label Jan 4, 2025
Copy link
Contributor

github-actions bot commented Jan 4, 2025

RSI Diff Bot; head commit 43c9677 merging into 00f3f68
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/ADT/Interface/Alerts/XenoQueenBloobCount.rsi

State Old New Status
power0 Added
power1 Added
power2 Added
power3 Added
power4 Added
power5 Added
power6 Added
power7 Added

Edit: diff updated after 43c9677

@NameLunar
Copy link
Contributor Author

И что это значит?
PoolManagerTestEventHandler: ERROR: Tests are taking too long. Shutting down all tests. This may lead to weird failures/exceptions.
Failed TestNoPendingDatabaseChanges [2 s]
Error Message:
System.Exception : CleanReturnAsync: Test killed the client in pair 4:

Copy link
Collaborator

@FaDeOkno FaDeOkno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

очень много лишнего, зачем делать всё на одной ветке?

_actionsSystem.RemoveAction(uid, component.ActionSpawnXenoRavager);
_actionsSystem.RemoveAction(uid, component.ActionSpawnXenoQueen);
}
private void OnCreateTurret(EntityUid uid, XenoQueenComponent component, InvisibleWallActionEvent args)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну какой InvisibleWallActionEvent

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ме.. Не хочу создавать новый пустой ивент..

@@ -294,6 +302,10 @@
- type: SlowOnDamage
speedModifierThresholds:
50: 0.7
# Start ADT: Xeno Buff
- type: StaminaDamageOnHit
damage: 60
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Окей. Уменьшить стоит. До 40

@@ -1165,7 +1165,7 @@
productEntity: ReinforcementRadioSyndicateCyborgAssault
icon: { sprite: Objects/Devices/communication.rsi, state: old-radio-borg-assault }
cost:
Telecrystal: 40 # ADT-Tweak 65>40
Telecrystal: 100 # ADT-Tweak 65>100
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

???

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Чекни, что с ботом сделал

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Эти изменения были где-то обговорены?
  2. Почему в этой ветке?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Борг имел просто 100 хп и все. 0 резисто. Играя на сервере многие пожаловались на то, что борг дно ибо КМ купил робота думаю "Это машина смерти! Она всех убьёт!", но после вопрос "Почему она откисла за 5 выстрелов?!"
  2. Из-за личный проблем я не могу выгружать новые ветки постоянно. Извините уж

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

???

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Меня попросили обновить шатл. Там Швабра (Маппер) сделал по лучше декали

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну так блин, а новый пр?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Почему в этой ветке?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я не имею возможности быстро переключаться на ветки и по этому мне не хочется это делать

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это связанно с плохим соединением и ошибкой загрузки


namespace Content.Shared.ADT.Events;

public sealed partial class SpawnWallActionEvent : InstantActionEvent
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Хорошо, ивент сделан, но почему он так называется?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ну.. Я там буду не только турели делать, а в принципе использовать. По задумке будут яйца, которые буду спавнить ксенов (Типа высаживать яйца и тд тп.) И чтобы их размещать я так же думал взять этот ивент

namespace Content.Shared.ADT.Events;

// TODO: This class needs combining with InstantSpawnSpellEvent
public sealed partial class SpawnXenoQueenEvent : WorldTargetActionEvent, ISpeakSpell
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Убери ISpeakSpell и добавь аттрибут DataDefinition. Будет то же самое, но без костылей

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Сделал

@NameLunar NameLunar changed the title [UPDATE Xeno and litle FIX Chaplain] [UPDATE Xeno and litle FIX] Jan 7, 2025
@github-actions github-actions bot removed the Changes: Map Изменение карт label Jan 9, 2025
Copy link
Collaborator

@FaDeOkno FaDeOkno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вот теперь хорошо

@Schrodinger71 Schrodinger71 merged commit 5ead783 into AdventureTimeSS14:master Jan 9, 2025
15 of 16 checks passed
@NameLunar NameLunar deleted the Draft branch January 9, 2025 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации Changes: Sprites Изменение спрайтов S: Needs Review S: Untriaged size/L Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants