Add: [bounty] Voice update #6326
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
завтра дошлифую |
Pelmenik123
changed the title
|
ссылка не на #отчёты-по-предложениям. |
BeebBeebBoob
added
the
Требует отчёт предложки
|
🤓 |
Antoonij
reviewed
Jan 8, 2025
Antoonij
reviewed
Jan 8, 2025
There was a problem hiding this comment.
В таком виде мержить не стоит из-за наличия большого количества костылей. Для приведения в минимальный, готовый к мержу вид стоит сделать следующее:
- Войсмодель нужно переделать на компонент, а не изобретать велосипед из проков, которые еще и вручную придется вызывать в разных местах. (RegisterWithParent - в нем регистрируют сигналы, Initialize компонента - тут будем уже деструктуризировать переданную куклу и получать голоса и другие варки)
- Убрать закомменченный код
- Переделать запоминание лиц при экзамайне на сигналы (COMSIG_MOB_RUN_EXAMINATE использовать)
|
бляяяяяяяяяяять |
|
сидеть еще компоненты курить |
BeebBeebBoob
added
Testmerge candidate
|
На всякий поставил лейбл кандидата. Такой ПР на вид конфликтный т.к. существует вероятность что откатят такое. |
BeebBeebBoob
requested changes
Jan 8, 2025
Den109G
reviewed
Jan 8, 2025
Den109G
reviewed
Jan 8, 2025
|
Костылей тут почти нет. Voice_model будет изменена на компоненты. Конфликтности не вижу на текущий момент (если новый сигнал фонда помощи уже в мердже то переделаю запоминание лиц) |
Den109G
reviewed
Jan 14, 2025
Den109G
reviewed
Jan 14, 2025
Den109G
reviewed
Jan 14, 2025
Den109G
reviewed
Jan 14, 2025
Den109G
requested changes
Jan 14, 2025
Co-authored-by: PlayerUnknown14 <[email protected]>
Den109G
reviewed
Jan 15, 2025
PlayerUnknown14
approved these changes
Jan 16, 2025
BeebBeebBoob
added
the
testmerge
BeebBeebBoob
removed
the
testmerge
|
седали пуки |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Описание
Добавляет новый датум voice_model. Название - ложь. Он служит для запоминания и вспоминания голосов (и лиц)
rename_self добавлен новый аргумент mass_update, он отвечает за то какие отделы запомнят новое имя челикса. Всем профессиям был указан департамент. Поскольку были сложности с изменением имен боргов и ИИ они теперь тупо на вайтлисте (ну, имена всех не хуманов достпуны и так, без запоминания). (саботеры будут жить). Добавлен новый параметр в mind - special_role_meta_know. Если он true то ты будешь всегда знать имена людей с такой же спец ролью. Обновлонее голоса перенесено из Life в say (нужно будет хорошенько потестить)
Причина создания ПР
https://discord.com/channels/617003227182792704/755125334097133628/1308878958451752981
Демонстрация изменений
Изменения для смертных: теперь ты будешь знать голоса только коллег из отдела и голоса глав. Чтобы запомнить новые голоса достаточно шифткликнуть. Клоуны и мимов корректно знают после выбора имени
Тесты
Боже, оно работает?