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.

Sign up for GitHub

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

Jump to bottom

New component AutoPostingChat(no Ready) #545

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

New component AutoPostingChat(no Ready) #545

wants to merge 16 commits into from

Conversation

Schrodinger71
Copy link
Collaborator

@Schrodinger71 Schrodinger71 commented Apr 1, 2024 •
edited
Loading

Описание PR

В данном PR добавлен новый компонент AutoPostingChat, который позволяет сущности автоматически постить сообщения в чат с эмоциями и IC разговорными фразами.

Добавлен чисто по фану

image

Демонстрация:

Устарело

output.compress-video-online.com.1.mp4

Проверки и таски

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Нужно чтобы поля SpeakTimerRead и EmoteTimerRead влияли на период отправки сообщений.
  • При удалении компонента прописать остановку таймеров.
  • Разделить Emote and Speak на 2 функции
  • Сделать флаги для рандомизации
  • Прописать поля для YAML файлов
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения
🆑 Schrodinger71

  • add: Новый компонент AutoPostingChat.

@Schrodinger71 Schrodinger71 added the DO NOT MERGE Не мерджить label Apr 1, 2024
@Schrodinger71 Schrodinger71 self-assigned this Apr 1, 2024
@Schrodinger71 Schrodinger71 reopened this Apr 1, 2024
@Schrodinger71 Schrodinger71 removed the DO NOT MERGE Не мерджить label Apr 6, 2024
@Schrodinger71
Copy link
Collaborator Author

Надо бы это закончить

@Schrodinger71 Schrodinger71 added the Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged. label Jun 12, 2024
@github-actions GitHub Actions
Copy link
Contributor

RSI Diff Bot; head commit 115cc61 merging into 7bf1fca
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

@github-actions github-actions bot removed the Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged. label Jun 14, 2024
@Schrodinger71 Schrodinger71 changed the title New component AutoPostingChat(В процессе) New component AutoPostingChat(Ready) Jun 14, 2024
modern-nm
modern-nm reviewed Jun 14, 2024
View reviewed changes
Copy link
Collaborator

@modern-nm modern-nm 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/ADT/AutoPostingChat/AutoPostingChatSystem.cs Outdated Show resolved Hide resolved
@Schrodinger71
Copy link
Collaborator Author

Если хочешь постить только эмоции, просто оставляешь поле пустым.... по идее

@Schrodinger71
Copy link
Collaborator Author

Хотя да, лучше поправить

@Schrodinger71 Schrodinger71 added the DO NOT MERGE Не мерджить label Jun 16, 2024
@Schrodinger71
Добавлен флаг рандомного интервала отправки сообщений в пределах 1-30…
3563f18 
… секунд, тернарные операторы для упрощения кода и описания объявлённых полей в компоненте.
@Schrodinger71
Добавил поля для YAML файлов.
f548521
@Schrodinger71
Разделил на функции
cc9b5c2
FaDeOkno
FaDeOkno reviewed Jun 20, 2024
View reviewed changes
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.

Кусаться буду

Content.Shared/ADT/AutoPostingChat/AutoPostingChatComponent.cs Outdated Show resolved Hide resolved
Content.Shared/ADT/AutoPostingChat/AutoPostingChatComponent.cs Outdated Show resolved Hide resolved
Content.Server/ADT/AutoPostingChat/AutoPostingChatSystem.cs
private System.Timers.Timer _speakTimer = new();
private System.Timers.Timer _emoteTimer = new();
private readonly Random _random = new Random();
//private static readonly Random _random = new Random();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Таймеры в системе? Обычно они находятся в компоненте, и не являются System.Timers.Timer, как и рандом.
Рандом используется как Dependency IRobustRandom, или чего-то такого. Посмотри примеры, пожалуйста

Content.Server/ADT/AutoPostingChat/AutoPostingChatSystem.cs Outdated Show resolved Hide resolved
Content.Shared/ADT/AutoPostingChat/AutoPostingChatComponent.cs Show resolved Hide resolved
Content.Server/ADT/AutoPostingChat/AutoPostingChatSystem.cs Outdated Show resolved Hide resolved
@Schrodinger71 Schrodinger71 added Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged. and removed Status: Needs Review labels Jun 20, 2024
@Schrodinger71 Schrodinger71 changed the title New component AutoPostingChat(Ready) New component AutoPostingChat(no Ready) Jun 20, 2024
@Mirokko
Copy link

Mirokko commented Sep 10, 2024

Как идея для развития:
Возможно следует сделать какуюнедь Map, чтоб привязать возможность работы компонента только в крите, или только когда жив, или только когда мёртв. Или несколько состояний сразу

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@modern-nm modern-nm modern-nm left review comments

@FaDeOkno FaDeOkno FaDeOkno left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@Schrodinger71 Schrodinger71

Labels
DO NOT MERGE Не мерджить Status: Awaiting Changes This PR needs its reviews addressed or changes to be made in order to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Schrodinger71 @Mirokko @modern-nm @FaDeOkno