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

Ai law update #435

Merged
merged 7 commits into from
Oct 27, 2024
Merged

Ai law update #435

merged 7 commits into from
Oct 27, 2024

Conversation

Ratyyy
Copy link
Collaborator

@Ratyyy Ratyyy commented Sep 8, 2024

Описание PR

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

Проверки

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

Изменения

🆑

  • add: ЦК приняли решение о введении на станцию плат для создания собственных наборов законов ИИ!
  • add: Ядра ИИ научились передавать свои законы боргам сразу после обновления законов!

@github-actions github-actions bot added the Changes: Localization Изменение локализации label Sep 8, 2024
Copy link
Collaborator

@KashRas2 KashRas2 left a comment

Choose a reason for hiding this comment

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

Первая часть ревью

Content.Server/Silicons/Laws/SiliconLawSystem.cs Outdated Show resolved Hide resolved
@github-actions github-actions bot added Status: Awaiting Changes Ожидание изменений and removed Status: Needs Review labels Sep 8, 2024
@Bolper Bolper requested a review from KashRas2 September 11, 2024 17:15
@github-actions github-actions bot added Status: Needs Review and removed Status: Awaiting Changes Ожидание изменений labels Sep 11, 2024
@Bolper
Copy link
Contributor

Bolper commented Sep 15, 2024

Неужели мне взятся за ревью

Copy link
Contributor

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

Copy link
Contributor

github-actions bot commented Oct 3, 2024

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

@Schrodinger71 Schrodinger71 dismissed KashRas2’s stale review October 27, 2024 09:03

Исправлено

@Schrodinger71 Schrodinger71 added this pull request to the merge queue Oct 27, 2024
Merged via the queue into master with commit 0d1eef0 Oct 27, 2024
13 checks passed
@Schrodinger71 Schrodinger71 deleted the ai-law-update branch October 27, 2024 09:04
@Schrodinger71 Schrodinger71 mentioned this pull request Nov 6, 2024
3 tasks
Schrodinger71 added a commit that referenced this pull request Nov 10, 2024
## Описание PR
<!-- Что вы изменили в этом пулл реквесте? -->

## Почему / Баланс
<!-- Почему оно было изменено? Ссылайтесь на любые обсуждения или
вопросы здесь. Пожалуйста, обсудите, как это повлияет на игровой баланс.
-->

## Техническая информация
<!-- Если речь идет об изменении кода, кратко изложите на высоком уровне
принцип работы нового кода. Это облегчает рецензирование.- -->


## Требования
<!--
В связи с наплывом ПР'ов нам необходимо убедиться, что ПР'ы следуют
правильным рекомендациям.

Пожалуйста, уделите время прочтению, если делаете пулл реквест (ПР)
впервые.

Отметьте поля ниже, чтобы подтвердить, что Вы действительно видели их
(поставьте X в скобках, например [X]):
-->
- [x] Я прочитал(а) и следую [Руководство по созданию пулл
реквестов](https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html).
Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению
мейнтейнера.
- [x] Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие
его изменения в игре, **или** этот пулл реквест не требует демонстрации
в игре

## Критические изменения 

- [ ] Проверить норм ли это робит
#435
по пути `Content.Server\Silicons\Laws\SiliconLawSystem.cs`
```cs
    protected override void OnUpdaterInsert(Entity<SiliconLawUpdaterComponent> ent, ref EntInsertedIntoContainerMessage args)
    {
        // TODO: Prediction dump this
        if (!TryComp(args.Entity, out SiliconLawBoundComponent? provider)) //ADT custom AI law
            return;

        var lawset = GetLaws(args.Entity, provider).Laws; //ADT custom AI law
        var query = EntityManager.CompRegistryQueryEnumerator(ent.Comp.Components);

        while (query.MoveNext(out var update))
        {
            SetLaws(lawset, update);
            if (provider.LawUploadSound != null && _mind.TryGetMind(update, out var mindId, out _))
                _roles.MindPlaySound(mindId, provider.LawUploadSound);
        }
        ///ADT AI Custom law start
        UpdateBorgsNTLaws(lawset);
    }
```

**Чейнджлог**

:cl:
 Шрёдька 
- tweak: Подтянуты коммиты с Корвакса, 06.11.2024.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Изменение локализации
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants