-
Notifications
You must be signed in to change notification settings - Fork 154
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
Languages #2226
base: master
Are you sure you want to change the base?
Languages #2226
Conversation
ГООООООООООООООООООЛ |
От себя добавлю, что логично было бы добавить дворфам владение Гаттерром, так как все дворфы из республики (откуда они все и идут) обязаны знать Гаттер для общения с республиканцами |
Самое время чистить акценты рас от всякой хуйни... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Прототипы посмотрел, вопрос только в реализации заменяемых слов. Они выполнены как я понял в виде листа и не особо хорошо читаются. У акцентов более гибкая структура прототипа, которую можно легко адаптировать. Обсудить с кирусом и сталеном.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А ХУ ЕТЬ
// If the original message is already there earlier encrypted, | ||
// it is taken from the cache, it is necessary for the correct display when sending in the radio, | ||
// when the character whispers and transmits a message to the radio | ||
if (ScrambleCache.TryGetValue(cacheKey, out var cachedValue)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хранить каждую фразу - плохая идея. Стоит посмотреть в сторону инъективного шифрования.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Хранить каждую фразу - плохая идея. Стоит посмотреть в сторону инъективного шифрования.
но там размера кэша из 20 сообщений всего и если он переполнен, то первое сообщение в листе кэша заменяется на новое... так ли это критично? просто я работу шифровальщика основывал на шифровальщике с пары и там так-же через кэш с лимитом 20 сообщений это работает...
if (!TryComp<LanguageComponent>(ent, out var comp) && _proto.TryIndex<LanguagesPrototype>("Universal", out _)) | ||
return true; | ||
|
||
if (comp != null && comp.CurrentLanguage == "Universal") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
у тебя же текущий язык содержится в листе LearnedLanguages
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
у тебя же текущий язык содержится в листе LearnedLanguages
но ведь при инициализации энтити с компонентом LanguageComponent и пустым листом LearnedLanguage универсальный язык присваивается в CurrentLanguage. Либо если добавить компонент LanguageComponent энтити у которого ранее этого компонента не было, то так же универсальный язык присваивается в CurrentLanguage, разве эта проверка тогда не нужна?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
но ведь при инициализации энтити с компонентом LanguageComponent и пустым листом LearnedLanguage универсальный язык присваивается в CurrentLanguage. Либо если добавить компонент LanguageComponent энтити у которого ранее этого компонента не было, то так же универсальный язык присваивается в CurrentLanguage, разве эта проверка тогда не нужна?
Почему бы в Learned не добавлять его тогда?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
но ведь при инициализации энтити с компонентом LanguageComponent и пустым листом LearnedLanguage универсальный язык присваивается в CurrentLanguage. Либо если добавить компонент LanguageComponent энтити у которого ранее этого компонента не было, то так же универсальный язык присваивается в CurrentLanguage, разве эта проверка тогда не нужна?
Почему бы в Learned не добавлять его тогда?
Чтобы лишний раз его не удалять из списка Learned админ командами, а просто в ВВ поменять Current на любой другой?
@Jopaglazik Стоило бы дойти до разработчиков и описать что хочешь делать. Пока концептуально есть проблема: я не могу написать следующим образом: ":ы ГСБ, этот :щ тупица :9 вокс, уже достал всех, выпишите ордер на него :щ иначе я разберусь с ним сам" |
@Jopaglazik подумай еще над реализацией замены частей сообщения (выделенного например с помощью тега, как предложил Алвур), а не всего сообщения. |
@Jopaglazik подумай еще над реализацией тир-1 СБ технологии на возможность перевода языков других рас в нормальное состояние, если заключенный отказывается говорить на общегалактическом, к примеру. |
Диктофон способный переводить в их распоряжении имеется, да и игра рольплейная, в крайнем случае пусть ищут переводчика что будет понимать заключенного, а наврёт ли переводчик или нет так-же решается рп процессом, вообщем пусть решают рп путём |
Насчёт шифрования части сообщения соглы надо подумать, но такое осилить разве что в будущем получится, лично я щас пока не вытяну, а касаемо лорных слов в зашифрованном сообщении - какой смысл? Носители языка будут видеть оригинальное сообщение, а не зашифрованное. Не носители будут видеть неразборчивый бред, зачем тогда не носителю в этом бреду искать контекст из "лорных" слов? Тем более это скорее часть акцентов, а не языка, как у дворфов например |
@Jopaglazik приветствую, как успехи? |
v draft poka zakinu |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
Описание PR
Добавление системы языков игру. Особо я свои позорные наработки не выкладывал, но у этой было большое желание довести её до ума т.к есть большая мотивация вайбить на таярочке, спасибо алвуру за то что их добавил иначе бы этого пра не было. Поэтому сквозь страдания, слезы, пот и бессонные дни с маниакальным рвением старался осуществить свою цель S-тира.
Основные изменения:
Минорные изменения
addlanguage <uid> <protoId>
removelanguage <uid> <protoId>
clearlanguages <uid>
Технические детали
AddLanguageSpecial
.LanguageComponent
присваивается "Универсальный" язык.Тестирование
Спасибо мелкому и альпаке за
детствотестированиеTTS
Так-как апи ТТСа у меня нету, на этом мои полномочия всё. Как он будет работать, будет ли он работать, насколько сильно будет нагружаться, насколько сильно захлёбываться. Возможно придётся санитайзить слога языков, чтобы произносилось нормально, если это вообще будет работать... или вообще придётся рефакторить много чего, тут я не знаю. Это решать уже ревьюверам что с этим думать.
Медиа
Проверки
Изменения
🆑 Jopaglazik